-
Notifications
You must be signed in to change notification settings - Fork 3.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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Reverting Anvil currently open modal state implementation #33067
Conversation
WalkthroughWalkthroughThe recent updates focus on removing features related to detached widgets in the Anvil layout system. This includes the elimination of specific Redux states, actions, and sagas associated with detached widgets. Additionally, there's an update in the CSS for widget overlay and a new selector function to handle widget IDs. The changes streamline the codebase by phasing out unused or redundant functionalities related to detached widgets. Changes
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (8)
Files skipped from review due to trivial changes (4)
Additional comments not posted (4)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Description
Tip
Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).
Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.
In #33040 we added a state exclusively to capture modals that are opened/closed to avoid computing which modal is open based on meta state.
However the problem with this approach that I had understood after testing it with deployed apps is that closing a modal is not done only via the saga, but is done via actions and the modal itself without dispatching the common saga.
Obviously the above implementation dunked.
Reverting to the previous implementation of relying on meta state to select visible detached widgets.
Fixes #
Issue Number
or
Fixes
Issue URL
Warning
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.
Automation
/ok-to-test tags="@tag.Anvil"
馃攳 Cypress test results
Tip
馃煝 馃煝 馃煝 All cypress tests have passed! 馃帀 馃帀 馃帀
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8895170744
Commit: 517ffc6
Cypress dashboard url: Click here!
Communication
Should the DevRel and Marketing teams inform users about this change?