-
Notifications
You must be signed in to change notification settings - Fork 35
Add 22ND1 and 22ND2 #412
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
Add 22ND1 and 22ND2 #412
Conversation
renetapopova
left a comment
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.
Thanks, @l-heemann. I added some editorial suggestions.
2afc27e to
c2273f6
Compare
renetapopova
left a comment
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.
I think all should be fine now.
|
@l-heemann, which version should those be merged into? |
@renetapopova The main feature has been merged to dev (2025.12.0) but it is behind a feature flag so customers can reach it at the earliest some time next year, unless they enable a feature flag. So far only a handful of customers know about the feature. 🤷 |
But given that nobody will see this error unless they are using the feature, I think it's safe to merge it when we publish 2025.12. |
Roles are not allowed to have denied privileges and auth rules at the same time
Applied straightforward suggestions, leaving some suggestions that require more thought Co-authored-by: Reneta Popova <reneta.popova@neo4j.com>
1665c6d to
72a0cbb
Compare
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
https://linear.app/neo4j/issue/COPS-301/update-to-notification-cannot-deny-with-auth-rule
The trello/linear template says
But I didn't find one for other codes?
Documents error codes introduced in https://github.com/neo-technology/neo4j/pull/33422