Skip to content

Commit

Permalink
Fix MPP-2857 - Add logic to prompt user to login if any authenticatio…
Browse files Browse the repository at this point in the history
…n information is missing (#487)

* Fix MPP-2857 - Add logic to prompt user to login if any authentication information is missing

* Fix MPP-2861 - Remove pointer-events from navigation tooltip
  • Loading branch information
maxxcrawford committed Apr 6, 2023
1 parent f5bc68a commit f5495ce
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/css/popup.css
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,7 @@
top: calc(100% + var(--spacingXs));
right: 0;
white-space: nowrap;
pointer-events: none;
}

.fx-relay-menu-dashboard-link:hover .fx-relay-menu-dashboard-link-tooltip,
Expand Down Expand Up @@ -247,7 +248,8 @@ sign-up-panel::after {
justify-content: center;
align-items: center;
text-align: center;
padding-top: var(--layoutSm);
/* Padding note: Bottom padding is necessary for bottom/fixed sign-in button */
padding: var(--layoutSm) 0 var(--layoutLg);
}

.fx-relay-sign-in-copy img {
Expand Down
9 changes: 9 additions & 0 deletions src/js/popup/popup.js
Original file line number Diff line number Diff line change
Expand Up @@ -1196,6 +1196,15 @@
userApiToken,
"apiToken"
);

// MPP-2857: During upgrade, the profile ID may be dropped, so if that is not
// available to the add-on, prompt the user to reauthenticate
const { profileID } = await browser.storage.local.get("profileID");

if (!profileID) {
return false;
}

return signedInUser;
},
getCachedServerStoragePref: async () => {
Expand Down

0 comments on commit f5495ce

Please sign in to comment.