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

Sign in with Google user registration notification does not always get dismissed #9933

Open
1 task
wpdarren opened this issue Dec 23, 2024 · 0 comments
Open
1 task
Assignees
Labels
Module: Sign in with Google Sign in with Google (SiwG) related issues. Next Up Issues to prioritize for definition P1 Medium priority Team S Issues for Squad 1 Type: Bug Something isn't working

Comments

@wpdarren
Copy link
Collaborator

wpdarren commented Dec 23, 2024

Bug Description

In edit mode, when the user has not enabled the user registration, a notification appears as expected but when I clicked on 'Got it' the notification remains. It's only when I click a second time that it gets dismissed.

It doesn't always happen though. it is a little sporadic in its behaviour.

Fast forward in the attached video to around 1:25 to see the issue in action.

Got.it.in.edit.mode.mov

Further context on dismissItem

It seems the dismissItem action being dispatched when the wp_googlesitekitpersistent_dismissed_items is either empty or removed causes two separate wp_googlesitekitpersistent_dismissed_items to be created. Two requests are made, one being legitimate and another being a bogus entry where slug is evaluated as true rather than a string:

Image
Image

If this request is processed by the server first, an empty wp_googlesitekitpersistent_dismissed_items is retrieved constantly:

Image

If it's not the first entry then everything is normal.


Do not alter or remove anything below. The following sections will be managed by moderators only.

Acceptance criteria

  • Only one request should be made when dispatching dismissItem, and it should be made where slug is a string.
  • Checks should be added to the dismissItem action such that only a string is a valid slug to prevent this issue in the future.

Implementation Brief

Test Coverage

QA Brief

Changelog entry

@wpdarren wpdarren added Module: Sign in with Google Sign in with Google (SiwG) related issues. P1 Medium priority Type: Bug Something isn't working labels Dec 23, 2024
@binnieshah binnieshah added the Next Up Issues to prioritize for definition label Jan 8, 2025
@tofumatt tofumatt self-assigned this Jan 9, 2025
@binnieshah binnieshah added the Team S Issues for Squad 1 label Jan 10, 2025
@tofumatt tofumatt assigned tofumatt and unassigned tofumatt Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Module: Sign in with Google Sign in with Google (SiwG) related issues. Next Up Issues to prioritize for definition P1 Medium priority Team S Issues for Squad 1 Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants