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

[UI] Saving an existent client #1771

Closed
helfi92 opened this issue Oct 28, 2019 · 3 comments · Fixed by #1811
Closed

[UI] Saving an existent client #1771

helfi92 opened this issue Oct 28, 2019 · 3 comments · Fixed by #1811
Assignees
Labels
bug Something isn't working good first issue Good for newcomers outreachy ui tc-web front end

Comments

@helfi92
Copy link
Contributor

helfi92 commented Oct 28, 2019

Reserved for outreachy candidates. In order for this to be assigned to you, make sure to complete the first step of the application process outlined in the project description.


To Reproduce
Steps to reproduce the behavior:

  1. Go to http://localhost:5080/auth/clients/email%2Fbastien%40nextcairn.com%2Ftc-notify-test
  2. Login with fake credentials (e.g., clientId test and accessToken test)
  3. Click on the "Delete on Expiration" switch

Actual behavior
The save button is not clickable after making a change to the "Delete on Expiration".

Screen Shot 2019-10-28 at 8 37 40 AM

Expected behavior
The save button should be clickable

Screen Shot 2019-10-28 at 8 37 45 AM

Additional context
The button becomes clickable when a user changes any of the other fields. It just doesn't work for the switch for some reason. Let's fix that.

@helfi92 helfi92 added bug Something isn't working good first issue Good for newcomers ui tc-web front end outreachy labels Oct 28, 2019
@edil-it-them
Copy link
Contributor

Can I work on this issue?

@djmitche
Copy link
Collaborator

@Rolikasi you mentioned it's hard working on multiple issues at once.. one way to avoid that is working on only one or two at a time :)

@km-js
Copy link
Collaborator

km-js commented Oct 28, 2019

I would like to work on this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers outreachy ui tc-web front end
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants