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

Sites dashboard: Clicking a site as non-admin leads to a broken flashing state #97109

Closed
fushar opened this issue Dec 5, 2024 · 3 comments · Fixed by #97172
Closed

Sites dashboard: Clicking a site as non-admin leads to a broken flashing state #97109

fushar opened this issue Dec 5, 2024 · 3 comments · Fixed by #97172
Assignees
Labels
[Experiment] AI labels added [Feature] Authentication Everything login and authentication. [Feature Group] User Management [Feature] My Home The main dashboard for managing a WordPress.com site. Needs triage Ticket needs to be triaged [Pri] Low Address when resources are available. [Status] Auto-allocated [Type] Bug When a feature is broken and / or not performing as intended

Comments

@fushar
Copy link
Contributor

fushar commented Dec 5, 2024

Quick summary

If I'm a non-admin of a site, and I click on that site from /sites, I'm supposed to get redirected to My Home immediately. However, I see a flashing screen of broken flyout panel.

Steps to reproduce

  1. Invite a test user to a site as Contributor / Editor.
  2. Log in as that user.
  3. Go to /sites.
  4. Click on that site.

What you expected to happen

I see My Home immediately.

What actually happened

I see a broken intermediate state:

Screen.Capture.on.2024-12-05.at.17-56-16.mp4

Impact

Some (< 50%)

Available workarounds?

There is no user impact

If the above answer is "Yes...", outline the workaround.

No response

Platform (Simple and/or Atomic)

No response

Logs or notes

It's similar to https://github.com/Automattic/dotcom-forge/issues/7311.

I think it's a regression when we decouple the sites dashboard logic from A4A.

@fushar fushar added [Type] Bug When a feature is broken and / or not performing as intended Needs triage Ticket needs to be triaged labels Dec 5, 2024
@fushar fushar self-assigned this Dec 5, 2024
@github-actions github-actions bot added [Pri] Low Address when resources are available. [Feature Group] User Management [Feature] Authentication Everything login and authentication. [Feature] My Home The main dashboard for managing a WordPress.com site. labels Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

OpenAI suggested the following labels for this issue:

  • [Feature Group] User Management: The issue is related to how non-admin users interact with site access and permissions.
  • [Feature] My Home: Since the issue revolves around the redirection to 'My Home' after clicking on a site, it's directly relevant.
  • [Feature] Authentication: The problem may involve user authentication when non-admin users attempt to access restricted areas.

@supernovia
Copy link
Contributor

I bumped into this while triaging The One Board. I'm setting this to In Progress; feel free to re-update the status if needed!

@okmttdhr
Copy link
Member

okmttdhr commented Dec 9, 2024

Thanks for fixing this, @fushar 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Experiment] AI labels added [Feature] Authentication Everything login and authentication. [Feature Group] User Management [Feature] My Home The main dashboard for managing a WordPress.com site. Needs triage Ticket needs to be triaged [Pri] Low Address when resources are available. [Status] Auto-allocated [Type] Bug When a feature is broken and / or not performing as intended
Development

Successfully merging a pull request may close this issue.

4 participants