Skip to content

Query Params Service Update#11

Open
NullVoxPopuli wants to merge 10 commits intorouter-computed-query-paramsfrom
query-params-service
Open

Query Params Service Update#11
NullVoxPopuli wants to merge 10 commits intorouter-computed-query-paramsfrom
query-params-service

Conversation

@NullVoxPopuli
Copy link
Owner

@NullVoxPopuli NullVoxPopuli commented Jun 9, 2020

@NullVoxPopuli
Copy link
Owner Author

NullVoxPopuli commented Jul 1, 2020

Notes / Ideas / Thoughts from meeting on 2020-07-01

Router is single source of truth of state model

  • state exposed via tracked
  • changed via transitionTo - no setter
    • including query params
  • setting a query param causes a router transition
  • eventually, transitions should be a queue?
    image

https://github.com/tc39/proposal-realms

  • Maybe route context is backed by a realm instead of dynamic scope?

Controller not needed for setting:
https://ember-twiddle.com/e5b7afe2bfcdea90ec3ba6f4d2c9dafa?openFiles=controllers.application%5C.js%2C&route=%2F%3Ffoo%3D123

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant