Commit fac904e
committed
fix(QueryStringManager): Could not subscribe to application.currentAppId$
Partially revert #9466. Current implementation doesn't work because
QueryStringManager is constructed during setup, and setApplication
happens during start. In playground or any instance we see `Could not
subscribe to application.currentAppId$` in console warning.
Signed-off-by: Joshua Li <[email protected]>1 parent 7f80625 commit fac904e
File tree
1 file changed
+10
-17
lines changed- src/plugins/data/public/query/query_string
1 file changed
+10
-17
lines changedLines changed: 10 additions & 17 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
48 | | - | |
49 | 48 | | |
50 | 49 | | |
51 | 50 | | |
| |||
58 | 57 | | |
59 | 58 | | |
60 | 59 | | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | 60 | | |
73 | 61 | | |
74 | 62 | | |
| |||
361 | 349 | | |
362 | 350 | | |
363 | 351 | | |
364 | | - | |
| 352 | + | |
| 353 | + | |
365 | 354 | | |
366 | | - | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
367 | 359 | | |
368 | 360 | | |
369 | | - | |
| 361 | + | |
370 | 362 | | |
371 | | - | |
372 | | - | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
373 | 366 | | |
374 | 367 | | |
375 | 368 | | |
| |||
0 commit comments