-
Notifications
You must be signed in to change notification settings - Fork 18
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
Fine grained access control #322
base: main
Are you sure you want to change the base?
Conversation
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
|
The Playwright tests fail because playwright doesn't use da-collab and da-admin from stage. They pass when I run them locally and once adobe/da-collab#61 and adobe/da-admin#108 are merged they will pass here too. |
Description
Add fine-grained access control as documented here: https://github.com/adobe/da-live/wiki/Using-ACLs
Performance characteristics can be found here: https://da.live/sheet#/da-sites/da-status/bosschae/performance/benchtimings
To try it out go here: https://acl--da-live--adobe.aem.page/?da-admin=stage&da-collab=stage
Test URLs:
Main: https://main--da-live--adobe.hlx.live/
Branch: https://acl--da-live--adobe.aem.page/?da-admin=stage&da-collab=stage
Related Issue
Fixes: adobe/da-admin#107
How Has This Been Tested?
Types of changes
Checklist: