-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Access control API docs #4922
Access control API docs #4922
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/keystonejs/keystone-next-docs/8kb1ojWRQmoUqspnUFkZNyCca95g |
|
6c20b46
to
1208df9
Compare
1208df9
to
7831dea
Compare
7831dea
to
0472a18
Compare
2a6a325
to
7c30119
Compare
@timleslie I wanted to remind you of condition in #3239 which was fixed in #3241, there was one condition left that it may still have incorrect sequence if user has no access to update some items (based on declarative control like isLocked = true) then the sequence in the resolveInput and afterChange would be different in the array. I can explain more if it does not remind you full context. |
7c30119
to
0a5b790
Compare
Thanks for the reminder of these. I'm going to follow this up with a separate PR to fix that behaviour. |
56c9535
to
c88366b
Compare
c88366b
to
4ffbfe4
Compare
Co-authored-by: Jed Watson <[email protected]>
Co-authored-by: Jed Watson <[email protected]>
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.
Nice!
Preview deployment here: https://keystone-next-docs-git-access-control-api-keystonejs.vercel.app/apis/access-control