Opened 7 months ago
Last modified 7 months ago
#7947 new enhancement
Remove the "remove" link if only one committers
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | low | |
| Component: | Plugin Directory | Keywords: | has-patch |
| Cc: |
Description
Hello,
On the advanced plugin page, we can manage the committers to the plugin.
On this plugin (one of mine) https://wordpress.org/plugins/connect-your-nuki-smartlock/advanced/, I'm the only one committer and I wonder if it is not a danger to have the availability to delete the only committer?
Change History (3)
This ticket was mentioned in PR #483 on WordPress/wordpress.org by @sebastienserre.
7 months ago
#2
- Keywords has-patch added; needs-patch removed
Updated logic to prevent committers from removing themselves. This ensures better user management.
Should fix https://meta.trac.wordpress.org/ticket/7947
7 months ago
#3
I don't think this is the optimal route, since there are times when you want to remove yourself from a plugin.
I'd probably only restrict it when you're the owner of the plugin, or you're the only committer of the plugin.
Perhaps this would be better implemented through a modal warning that they're about to remove themselves.
As far as I know, you can't actually remove yourself if you're the only committer, but having the link there is indeed not ideal.
https://github.com/WordPress/wordpress.org/blob/d6ad55185e2550457b287aa5651e25f395f0c8fe/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin-committers.php#L143-L149