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.

Sign up for GitHub

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

Jump to bottom

CES Prompts Showing several times #1428

Closed
puntope opened this issue Apr 15, 2022 · 1 comment · May be fixed by #1543
Closed

CES Prompts Showing several times #1428

puntope opened this issue Apr 15, 2022 · 1 comment · May be fixed by #1543
Assignees
@jorgemd24
Labels
priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.

Comments

@puntope
Copy link
Contributor

puntope commented Apr 15, 2022 •
edited
Loading

Describe the bug:

While I was testing other PR I noticed the CES Prompt "How easy was it to create a Google Ad campaign?" (maybe some other Prompts as well) are being shown multiple times.

In my opinion, if the Prompt is already there, we should avoid showing it again.

Screen.Recording.2022-04-15.at.11.37.21.mov

Steps to reproduce (actual behaviour):

  1. WP Admin as well as Analytics Feature should be enabled
  2. Edit a campaign and save
  3. You will see the prompt "How easy was it to create a Google Ad campaign?"
  4. Dont do anything and just go to Edit a campaign and save again
  5. You will see another prompt "How easy was it to create a Google Ad campaign?" so 2 prompts in total.

Expected behavior:

  1. WP Admin as well as Analytics Feature should be enabled
  2. Edit a campaign and save
  3. You will see the prompt "How easy was it to create a Google Ad campaign?"
  4. Dont do anything and just go to Edit a campaign and save again
  5. You WONT see another prompt "How easy was it to create a Google Ad campaign?" so 1 prompt in total.
@puntope puntope added type: bug The issue is a confirmed bug. priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. labels Apr 15, 2022
@jconroy
Copy link
Member

jconroy commented May 30, 2022

@jorgemd24 could you pick this one up too please?

@jorgemd24 jorgemd24 mentioned this issue May 31, 2022
Open
@jorgemd24 jorgemd24 mentioned this issue Jun 14, 2022
Merged
@puntope puntope closed this as completed Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jorgemd24 jorgemd24

Labels
priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.
Projects
None yet
Milestone
No milestone
3 participants
@jconroy @jorgemd24 @puntope