Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix pre-complete Access Token permissions #33691

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

Gsvd
Copy link
Contributor

@Gsvd Gsvd commented Feb 23, 2025

Fixes #33519

This PR allows the backend to send JSON back to the UI to pre-fill the previously selected permissions.

Hey, sorry—I only realized after I finished that someone had already made a suggestion (#33614).

Anyway, I'm sending mine in case my solution is preferred.
I also saw the comments about removing the Vue component; it's true that it's not very useful...

I'll check on my end to see if I can make it work.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 23, 2025
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 23, 2025
@github-actions github-actions bot added modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/frontend labels Feb 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/frontend modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UX,Usability: Creating Personal Access Tokens, provide access form values on error
2 participants