Skip to content
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

Immutable roles #1984

Closed
wants to merge 4 commits into from
Closed

Immutable roles #1984

wants to merge 4 commits into from

Conversation

phil198
Copy link
Contributor

@phil198 phil198 commented Nov 25, 2024

adds instructions and examples for how to administer immutable roles which are to be used in conjunction with immutable privileges.

Combines the immutable roles docs with those for immutable privileges because the two are part of the same feature.

@mnd999 mnd999 self-assigned this Nov 26, 2024
Copy link
Contributor

@mnd999 mnd999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nits. Also I think the convention is one sentence per line so you might need to reformat in a few places.

@phil198 phil198 requested a review from mnd999 November 27, 2024 13:34
Copy link
Contributor

@mnd999 mnd999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems okay to me now

@neo-technology-commit-status-publisher
Copy link
Collaborator

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

[[access-control-immutable-roles]]
=== Immutable roles
Immutable roles are roles which cannot be modified in the usual way.
This means they cannot be created, renamed, dropped, or have privileges granted to or revoked from them.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This contradicts the above commands, for example, CREATE [IMMUTABLE] ROLE name AS COPY OF otherName or CREATE OR REPLACE [IMMUTABLE] ROLE name [AS COPY OF otherName], etc.

@renetapopova
Copy link
Contributor

Replaced by #2001

renetapopova added a commit that referenced this pull request Dec 4, 2024
Replaces #1984

---------

Co-authored-by: Phil Wright <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants