75599f8847
## Description 1. After releasing the migration from policies to policyMap we have seen instances where customers went back to previous versions and faced issues around missing permissions. 2. In another instance we ended up in a state where the datasource permission were missing. While triaging these issues we couldn't able to figure out the root cause so decided to keep both policies and policyMap in the DB. This will rule out the issue if user went back to previous versions and upgrades to latest we will still have some reference to look at. Ref: https://theappsmith.slack.com/archives/C02K2MZERSL/p1724779433650089 /test all ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/10716193933> > Commit: bbfa71e3b4940de7d98b969a65285c2d03e611a1 > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=10716193933&attempt=1" target="_blank">Cypress dashboard</a>. > Tags: `@tag.All` > Spec: > <hr>Thu, 05 Sep 2024 08:52:20 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [x] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Summary by CodeRabbit - **New Features** - Enhanced policy management by transitioning from the `policies` field to a more efficient `policyMap`. - **Bug Fixes** - Improved state management during the pristine state reset, ensuring policies are cleared correctly. - Strengthened the removal process for policies to prevent concurrent modification issues. - **Chores** - Optimized policy handling for better performance and readability by reducing redundant method calls. <!-- end of auto-generated comment: release notes by coderabbit.ai --> |
||
|---|---|---|
| .. | ||
| src | ||
| auth-notes | ||
| pom.xml | ||