feat(admin): user select modal for bulk transfer + author updated on transfer - #6
Open
debba wants to merge 2 commits into
Open
feat(admin): user select modal for bulk transfer + author updated on transfer#6debba wants to merge 2 commits into
debba wants to merge 2 commits into
Conversation
Replace the window.prompt asking for a raw user ID with a proper modal: the new owner is picked from a select listing all users (display name, email, admin marker), loaded once from the admin users endpoint and sorted by name. The confirm button stays disabled until a user is chosen and the modal only closes when the transfer succeeds. Adds a generic Modal component (bits-ui Dialog) alongside the destructive-only ConfirmDialog, for dialogs that carry a form instead of a typed confirmation word.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
The author column is written once at submit time (provider username + repo URL) and nothing ever touched it again, so after any ownership transfer the catalog and detail pages kept crediting the previous owner. All three transfer paths — admin bulk transfer, admin single-plugin PATCH, and the recipient accepting a consensual transfer — now rewrite author to '<new owner display name> <repoUrl>', the same shape the OAuth submit flow produces. The bulk path updates per plugin since the string embeds each plugin's repo URL.
debba
force-pushed
the
feat/transfer-user-select
branch
from
August 8, 2026 06:19
7b3fe09 to
123cf1b
Compare
debba
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Follow-ups to the bulk ownership transfer released in v0.13.5.
User select instead of a raw ID prompt
The Transfer bulk action asked for the new owner's user ID via
window.prompt. It now opens a modal where the new owner is picked from a select listing all users (display name, email, admin marker), loaded once fromGET /api/admin/usersand sorted by name. The confirm button is disabled until a user is chosen, and the modal closes only when the transfer succeeds.This adds a generic
Modalcomponent (bits-uiDialog) alongside the existingConfirmDialog, which is specialized for destructive type-to-confirm flows.Author line updated on ownership transfer
The
authorcolumn is written once at submit time (provider username + repo URL) and nothing ever touched it again, so after a transfer the catalog cards and the plugin detail page kept crediting the previous owner.All three transfer paths — admin bulk transfer, admin single-plugin
PATCH, and the recipient accepting a consensual transfer — now rewriteauthorto<new owner display name> <repoUrl>, the same shape the OAuth submit flow produces. The bulk path updates per plugin since the string embeds each plugin's repo URL. No frontend or schema changes needed.New messages (
admin_plugins_bulk_transfer_title/description/owner_label/placeholder) are translated in all six locales; the oldadmin_plugins_bulk_transfer_promptkey is removed.