-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(editor): Migrate ui.store
to use composition API (no-changelog)
#9892
Conversation
ui.store
to composition APIui.store
to use composition API (no-changelog)
3 flaky tests on run #5770 ↗︎
Details:
5-ndv.cy.ts • 2 flaky tests
10-undo-redo.cy.ts • 1 flaky test
Review all test suite changes for PR #9892 ↗︎ |
- delete isNodeView computed method as it's not used anymore. - Remove of instances of isAnyModal open as we do not have that method in the store
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good. Same comments about value caching like the previous PR.
I completely agree about the splitting of this store! We could tackle that in a separate PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for addressing all the feedback! 💪
|
✅ All Cypress E2E specs passed |
Summary
ui.store
to composition API.isNodeView
andareExpressionsDisabled
.isPageLoading
andmainPanelPosition
from the state as they are not used anymore.Clean up all the uses ofisAnyModalOpen
as we do not have that method in the store.@alexgrozav thinking a bit further:
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)