Skip to content

permission matrix manipulation#434

Merged
andreasfrisch merged 5 commits into
developfrom
feature/manage-permission-matrix
May 12, 2026
Merged

permission matrix manipulation#434
andreasfrisch merged 5 commits into
developfrom
feature/manage-permission-matrix

Conversation

@andreasfrisch
Copy link
Copy Markdown
Contributor

@andreasfrisch andreasfrisch commented Apr 28, 2026

🌟 Changes:

  • Adds endpoint for any authenticated user to fetch the current RBAC rules
  • Adds endpoint for cloudengineers (outside RBAC) to manipulate RBAC rules

🎶 Notes:
The new endpoint for getting rbac/permission-matrix sort of supercedes the existing one for rbac/get-assignable-permissions. We should at least keep the old one until the frontend is updated.

It feels weird to have something called Bootstrap that is used for more than just bootstrapping 🫣

❓ Question:
It seems okay that we have to update one role at a time when updating permissions.
However, it feels a little clunky in the frontend.
Should we simply allow PUTting all the permissions into the system in one fell swoop?

@DFDS-Snyk
Copy link
Copy Markdown

DFDS-Snyk commented Apr 28, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@andreasfrisch andreasfrisch requested a review from SEQUOIIA April 28, 2026 14:48
@andreasfrisch andreasfrisch marked this pull request as draft April 28, 2026 14:58
@andreasfrisch andreasfrisch force-pushed the feature/manage-permission-matrix branch from c67e7ff to 5588849 Compare May 6, 2026 16:01
@andreasfrisch andreasfrisch marked this pull request as ready for review May 6, 2026 16:02
@andreasfrisch andreasfrisch merged commit d24abd6 into develop May 12, 2026
9 checks passed
@andreasfrisch andreasfrisch deleted the feature/manage-permission-matrix branch May 12, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants