Skip to content

Allow (Domain-) Admins to delete aliases#873

Merged
t2d merged 5 commits intomainfrom
admin-delete-alias
Jan 31, 2026
Merged

Allow (Domain-) Admins to delete aliases#873
t2d merged 5 commits intomainfrom
admin-delete-alias

Conversation

@t2d
Copy link
Copy Markdown
Contributor

@t2d t2d commented Sep 4, 2025

Admin:
image

User:
image

@t2d t2d force-pushed the admin-delete-alias branch from 0b09a08 to 68a1088 Compare September 4, 2025 20:18
@t2d t2d marked this pull request as ready for review September 4, 2025 20:19
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 4, 2025

Comment thread src/Voter/AliasVoter.php Outdated
@t2d t2d force-pushed the admin-delete-alias branch from 68a1088 to 67bf3a6 Compare January 30, 2026 06:04
Comment thread src/Voter/AliasVoter.php Fixed
Comment thread src/Voter/AliasVoter.php Fixed
Comment thread src/Voter/AliasVoter.php Fixed
t2d and others added 3 commits January 30, 2026 07:06
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t2d t2d requested a review from 0x46616c6b January 30, 2026 06:19
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@t2d t2d force-pushed the admin-delete-alias branch from b9cea85 to 352a71c Compare January 30, 2026 06:31
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@0x46616c6b 0x46616c6b left a comment

Choose a reason for hiding this comment

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

Only open question (but not related to this change): When you delete a shared alias, will only the reference to the current user be deleted or the whole alias?

@t2d
Copy link
Copy Markdown
Contributor Author

t2d commented Jan 31, 2026

The database doesn't know shared aliases. These are simply multiple aliases with the same source.

@t2d t2d merged commit 0cebb5e into main Jan 31, 2026
9 checks passed
@t2d t2d deleted the admin-delete-alias branch January 31, 2026 07:24
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