-
Notifications
You must be signed in to change notification settings - Fork 4
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
Users page: copy user email addresses #1660
base: nextjs
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for vision-zero-nextjs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for vision-zero-nextjs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -48,21 +68,29 @@ export default function Users() { | |||
<Button | |||
className="me-2" | |||
onClick={() => setShowNewUserModal(true)} | |||
disabled={isValidating} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
disable the buttons while we are showing the isValidating
spinner (different from the isLoading
spinner which is for loading all the users in the table)
Associated issues
Closes cityofaustin/atd-data-tech#20121
Testing
URL to test:
Local - you will need to run the users api and also update
isValidUser
inapi/server.py
to return true until john gets rid of that functionSteps to test:
isValidating
is true there will be a spinner by the buttons and they should be disabled.Ship list
main
branch