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

DEVOPS-1665: Minor mustache template updates #423

Closed
wants to merge 1 commit into from

Conversation

MrCreosote
Copy link
Member

@MrCreosote MrCreosote commented Feb 14, 2024

  • Make it easier to see the entire URL in the config page
  • Make the revoke all tokens button scarier

Closes #344

image
image

* Make it easier to see the entire URL in the config page
* Make the revoke all tokens button scarier
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

Merging #423 (fbb6119) into develop (c7d7ce3) will not change coverage.
Report is 3 commits behind head on develop.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #423   +/-   ##
==========================================
  Coverage      93.39%   93.39%           
  Complexity      2142     2142           
==========================================
  Files            126      126           
  Lines           7520     7520           
  Branches        1178     1178           
==========================================
  Hits            7023     7023           
  Misses           454      454           
  Partials          43       43           

to error out. No warning is given; if you click the button below it happens.</p>
<form action="{{revokeallurl}}" method="post">
<input style="background-color:red;color:white" type="submit"
value="REVOKE EVERY SINGLE TOKEN"/>

Choose a reason for hiding this comment

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

This might benefit from an extra line or two to clarify things -- e.g. "IDGAF, REVOKE EVERY SINGLE TOKEN, MOFO!"; "REVOKE EVERY SINGLE TOKEN. MUAHAHAHAHA!"; "WHO NEEDS A SOUL ANYWAY? REVOKE EVERY SINGLE TOKEN!"

@MrCreosote MrCreosote closed this Feb 14, 2024
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.

Document how to set up alternate environments more clearly
2 participants