Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

addon installed state races on home page #1606

Closed
dannycoates opened this issue Oct 13, 2016 · 4 comments
Closed

addon installed state races on home page #1606

dannycoates opened this issue Oct 13, 2016 · 4 comments
Assignees
Labels
Milestone

Comments

@dannycoates
Copy link

I managed to trigger two unfortunate races setting the installed state of the addon that caused the experiment list page to show the no-addon version instead of the (correct) with-addon version. Once the page was in the no-addon state it stayed there.

I added a few console.warn lines before each dispatch of setHasAddon. Here's the log from a time when the page goes from the loading view to no-addon view.

screen shot 2016-10-13 at 10 42 58 am

And here's a particularly weird one where the page goes from loading flashing to with-addon but then quickly to no-addon.

screen shot 2016-10-13 at 11 06 43 am

@dannycoates dannycoates self-assigned this Oct 13, 2016
@dannycoates
Copy link
Author

Happens in prod too, so it wasn't added this week

@dannycoates dannycoates added this to the Sprint 12 milestone Oct 13, 2016
@dannycoates
Copy link
Author

This is PageMod being a jerk. The short of it is that the detach event from a previous page load can trigger on the current page and trick the frontend into thinking the addon is gone.

It doesn't look like a "normal" page refresh is problematic so far, but when you navigate by changing the url (or not) in the address bar and hitting enter there's a chance you'll get this behavior.

@SoftVision-PaulOiegas
Copy link

Isn't this the same issue from #1574 ?

@dannycoates
Copy link
Author

Isn't this the same issue from #1574 ?

Indeed it is!

dannycoates added a commit to dannycoates/testpilot that referenced this issue Oct 14, 2016
dannycoates added a commit to dannycoates/testpilot that referenced this issue Oct 14, 2016
dannycoates added a commit to dannycoates/testpilot that referenced this issue Oct 17, 2016
lmorchard pushed a commit to lmorchard/testpilot that referenced this issue Oct 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
No open projects
Development

No branches or pull requests

3 participants