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

When you enter the wrong username/password combination, the login dropdown turns blank #2515

Closed
artlowel opened this issue Sep 25, 2023 · 5 comments · Fixed by #2530
Closed
Assignees
Labels
Milestone

Comments

@artlowel
Copy link
Member

Describe the bug
If you enter the wrong combination of username and password in the login dropdown, or on the login page. The component turns blank entirely. I can reproduce this on https://demo.dspace.org and on https://sandbox.dspace.org

image

To Reproduce
Steps to reproduce the behavior:

  1. Click the "log in" link in the header
  2. Enter a username and password combination that doesn't exist
  3. Click the "log in" button

Expected behavior
It should show an error message and allow you to try again.

Related work
Possibly related to #2414?

@artlowel artlowel added bug needs triage New issue needs triage and/or scheduling labels Sep 25, 2023
@github-project-automation github-project-automation bot moved this to 🆕 Triage in DSpace Backlog Sep 25, 2023
@artlowel
Copy link
Member Author

The reason seems to be this d-none. While the idea is good, it looks like contentRef doesn't update here when you get an error

@tdonohue
Copy link
Member

@artlowel : Thanks for figuring this out. Assigning back to you for you or your team to work on. Also pinging @alexandrevryghem who added that code.

@alanorth
Copy link
Contributor

I see the empty login dropdown very often after refreshing the page. I haven't yet figured out why, but suspect it happens when reloading too soon after restarting the backend or frontend in dev mode. I am used to shift-refreshing the home page to fix it...

@tdonohue
Copy link
Member

I've found that I'm able to reproduce this in Firefox but NOT in Chrome. In Firefox, I see the exact behavior shown in the description (entire login dropdown turns blank). In Chrome though, the login dropdown flashes blank but then shows the error message as expected. Nonetheless, it seems like it affects some browsers.

I'll test the fix PR next and see if it resolves the issue for me in Firefox.

@tdonohue tdonohue added this to the 7.6.1 milestone Oct 25, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In Progress to ✅ Done in DSpace 8.x and 7.6.x Maintenance Oct 25, 2023
@alanorth
Copy link
Contributor

alanorth commented Oct 25, 2023

@tdonohue ah that explains it. I use Firefox explicitly and see the blank login drop down so frequently that I was surprised nobody had noticed yet (it's certainly not only limited to a wrong username/password combination).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
3 participants