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

Put hint on email fields, etc, when editing user not to prefill #1165

Closed
ymnoor21 opened this issue May 2, 2021 · 6 comments · Fixed by #1169
Closed

Put hint on email fields, etc, when editing user not to prefill #1165

ymnoor21 opened this issue May 2, 2021 · 6 comments · Fixed by #1169
Labels
Milestone

Comments

@ymnoor21
Copy link

ymnoor21 commented May 2, 2021

Describe the bug

I am one of the admins of my VA. So when trying to approve a pilot application by editing the user, I see my email address in the user editor screen.

Version

version 7.0.0-dev

To Reproduce

Steps to reproduce the behavior:

  1. You need to have a pending pilot application.
  2. VIsit: https://<yourdomain.com>/admin/users?search=state:0
  3. Edit the user.
  4. Check the email address and password field. It shows the admin email, not the user email.

Expected behavior

This should show the user's email and not the admin email. Moreover, the password field shouldn't be here. Users already have that functionality from the login screen with the "Forget Password" link. Password change should not be an admin functionality.

Better yet, add an approve button on the user list page. You can always go edit the user to set a different status.

Screenshots

I cannot share the screenshot because of the user's data privacy.

@FatihKoz
Copy link
Contributor

FatihKoz commented May 3, 2021

I tried to replicate the issue, it works as it should ... I see the user email and not admin email.

Registered with a fake/test user to my test server;

01

And clicked edit to see the result;

02

It is the email I used during registration, not my admin e-mail.

Moreover, the password field shouldn't be here.

And I think it should be there, it does not show the user's password BUT will give admins the ability to assist users if they are not able to change their password etc for some reason or when it is really needed. So in my opinion it should be there and that should be an admin functionality too.

Btw, latest dev version is 7.0.0-dev+210423.ec6cb4 dated 23 APR 2021 ... I assume you are using that too (and I never saw my admin email during approval of users)

@nabeelio
Copy link
Owner

nabeelio commented May 3, 2021

Hmm yeah I'm not seeing my admin email there either on the demo. Strange. The password field will be there since the admin can reset it to something if needed

@FatihKoz
Copy link
Contributor

FatihKoz commented May 3, 2021 via email

@ymnoor21
Copy link
Author

ymnoor21 commented May 3, 2021

Guys, this might be an issue with my browser prefilling it. So if you have used the browser option to save the username and password for the login screen, it shows up in the user edit screen. I tried incognito and found the issue, otherwise, it didn't show the admin email. So it may not be an issue after all. Sorry about this. Please close this issue if you want.

@FatihKoz
Copy link
Contributor

FatihKoz commented May 3, 2021 via email

@nabeelio nabeelio added the bug label May 3, 2021
@nabeelio nabeelio added this to the 7.0.0 milestone May 3, 2021
@nabeelio nabeelio changed the title Pending approval screen show admin email Put hint on email fields, etc, when editing user not to prefill May 3, 2021
@nabeelio
Copy link
Owner

nabeelio commented May 3, 2021

Ah, so I changed the title of this, I think then I have to add something to those input boxes so that the browser won't autofill those.

nabeelio pushed a commit that referenced this issue May 4, 2021
* Disable autocomplete on admin email field

Fixes #1165

* Disable autocomplete on admin name and password

For sake of completeness and due to some over zealous autocompleters, also disable it on name and password fields
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants