Skip to content

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

neo-technology-commit-status-publisher commented Dec 3, 2024

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
Collaborator

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
Collaborator

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