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

Generate and revoke access tokens with standard UI components #1597

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

mawinter69
Copy link
Contributor

small UI update for the access tokens
replace yui-button with jenkins-button to be in sync with the overall jenkins styling

Use a dialog to confirm revocation of access tokens

Before:
image

After:
image
image

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. This is simply a reminder of what we are going to look for before merging your code. If a checkbox or line does not apply to this pull request, delete it. We prefer all checkboxes to be checked before a pull request is merged

  • I have read the CONTRIBUTING doc
  • Unit tests pass locally with my changes
  • No spotbugs warnings were introduced with my changes
  • I have interactively tested my changes

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply. Delete the items in the list that do not apply

  • New feature (non-breaking change which adds functionality)

Further comments

If this is a relatively large or complex change, start the discussion by explaining why you chose the solution you did and what alternatives you considered.

small UI update for the access tokens
replace yui-button with jenkins-button to be synv with the overall
jenkins styling
Use a dialog to confirm revocation of access tokens
@mawinter69 mawinter69 requested a review from a team as a code owner June 15, 2024 23:29
@MarkEWaite MarkEWaite changed the title use jenkins buttons and dialog Use standard dialog and buttons to generate and revoke access tokens Jun 15, 2024
@MarkEWaite MarkEWaite changed the title Use standard dialog and buttons to generate and revoke access tokens Generate and revoke access tokens with standard UI components Jun 16, 2024
@MarkEWaite MarkEWaite added the rfe Improvement or new feature label Jun 16, 2024
@MarkEWaite MarkEWaite merged commit b71dd01 into jenkinsci:master Jul 26, 2024
15 checks passed
MarkEWaite added a commit to MarkEWaite/docker-lfs that referenced this pull request Jul 29, 2024
Incremental that includes:

* Use poll env when checking for ls-remote branches
  jenkinsci/git-plugin#1511

* Optionally report the most recent commit as the changelog of a new build
  jenkinsci/git-plugin#1565
  https://issues.jenkins.io/browse/JENKINS-4162
  https://issues.jenkins.io/browse/JENKINS-14138
  https://issues.jenkins.io/browse/JENKINS-26354
  https://issues.jenkins.io/browse/JENKINS-68030

* Generate and revoke access tokens with standard UI components
  jenkinsci/git-plugin#1597

* Do not assume HttpResponse is a functional interface
  jenkinsci/git-plugin#1590
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rfe Improvement or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants