Skip to content

MCR-3753 Add command to delete all permissions#3029

Open
toKrause wants to merge 1 commit into
mainfrom
issues/MCR-3753_Add-command-to-delete-all-permissions
Open

MCR-3753 Add command to delete all permissions#3029
toKrause wants to merge 1 commit into
mainfrom
issues/MCR-3753_Add-command-to-delete-all-permissions

Conversation

@toKrause

@toKrause toKrause commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Link to jira.

Pull Request Checklist (Author)

Please go through the following checklist before assigning the PR for review:

Ticket & Documentation

  • The issue in the ticket is clearly described and the solution is documented.
  • Design decisions (if any) are explained.
  • The ticket references the correct source and target branches.
  • The fixed-version is correctly set in the ticket and matches the PR's target branch (main).

Feature & Improvement Specific Checks

  • Instructions on how to test or use the feature are included or linked (e.g. to documentation).
  • For UI changes: before & after screenshots are attached.
  • New features or migrations are documented.
  • Does this change affect existing applications, data, or configurations?
    • Yes: Is a migration required? If yes, describe it.
    • Breaking change is marked in the commit message.

Testing

  • I have tested the changes locally.
  • The feature behaves as described in the ticket.
  • Were existing tests modified?
    • Yes: explain the changes for reviewers.

MCR Conventions & Metadata

  • MCR naming conventions are followed
  • If the public API has changed:
    • Old API is deprecated or a migration is documented.
    • If not, no action needed.
  • Java license headers are added where necessary.
  • Javadoc is written for non-self-explanatory classes/methods (Clean Code).
  • All configuration options are documented in Javadoc and mycore.properties.
  • No default properties are hardcoded — all set via mycore.properties.

Multi-Repo Considerations

  • Is an equivalent PR in MIR required?
    • If yes, is it already created?

@toKrause
toKrause force-pushed the issues/MCR-3753_Add-command-to-delete-all-permissions branch 3 times, most recently from 5314440 to 2f617b3 Compare July 6, 2026 12:55
@toKrause toKrause removed the postponed label Jul 6, 2026
@toKrause
toKrause force-pushed the issues/MCR-3753_Add-command-to-delete-all-permissions branch 3 times, most recently from 0bebfa4 to f84becc Compare July 14, 2026 13:11
@toKrause
toKrause requested a review from rsteph-de July 14, 2026 13:11
@toKrause

Copy link
Copy Markdown
Contributor Author

@rsteph-de I tested the "new and extended" implementation and it works as intended:

image image

@toKrause
toKrause force-pushed the issues/MCR-3753_Add-command-to-delete-all-permissions branch from f84becc to 7e44226 Compare July 14, 2026 14:10
@toKrause

Copy link
Copy Markdown
Contributor Author

However, deleting the rules can cause problems. Adding new rules again only works from the CLI, not the WebCLI.
Therefore, I added an additional command that only deletes the rule mapping, not the rules.

@toKrause
toKrause force-pushed the issues/MCR-3753_Add-command-to-delete-all-permissions branch 2 times, most recently from fc01c2f to 30455d3 Compare July 21, 2026 16:07
@toKrause
toKrause force-pushed the issues/MCR-3753_Add-command-to-delete-all-permissions branch from 30455d3 to fe322ec Compare July 21, 2026 20:28
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.

1 participant