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

You are wrongly logged out from "Firefox Notes" after reopening the sidebar if a sync action was previously performed #802

Closed
Softvision-MariusComan opened this issue Mar 26, 2018 · 5 comments
Assignees
Labels
ARCHIVED CLOSED at time of archiving
Milestone

Comments

@Softvision-MariusComan
Copy link

[Affected versions]:

  • Firefox 59.0.1 and above

[Affected Platforms]:

  • All Windows
  • All Mac
  • All Linux

[Prerequisites]:

  • Have a Firefox profile with the latest version of the "Firefox Notes" add-on (3.2.0dev custom built on 2018-03-26) installed.
  • You are logged into "Firefox Notes".
  • Have ~20 notes in the sidebar.

[Steps to reproduce]:

  1. Open the browser with the profile from prerequisites.
  2. Click the "Sync Your Notes" button.
  3. Close and reopen the sidebar.
  4. Observe the bottom toolbar.

[Expected result]:

  • The "Synced at (hour) message is displayed.

[Actual result]:

  • You are logged out and the "Sign in to Sync" message is displayed.

[Notes]:

  • The issue is also reproducible if the sidebar is switched/closed while the "Syncing changes..." message is displayed.
  • The issue is still reproducible after restart.
  • Attached a screen recording of the issue:
    rec of the issue
@vladikoff vladikoff added this to the Triaged milestone Mar 26, 2018
@vladikoff
Copy link
Contributor

Right, if there is an error with sync we don't show an error and then just lose the state, Ref: #791 (comment)

@sebastienbarbier
Copy link
Collaborator

@ryanfeeley can you suggest a UI for an error notification ?

We asked ourself where an alert would shows up :
alert
However I don't think we can change the title: "The page at moz-extension:xxx-xxxx..."

@vladikoff
Copy link
Contributor

@ryanfeeley can we reuse footer notifications for this, and could you please provide text/copy for this?

@ryanfeeley
Copy link
Contributor

ryanfeeley commented Mar 28, 2018

@vladikoff Something like:

Oops! Something went wrong with syncing your Firefox Notes, and you were signed out. Sign in to Sync from within Notes to resume syncing.
OK

vladikoff pushed a commit that referenced this issue Mar 29, 2018
After authenticating, kinto might return some errors we should handle.

 fix synced UI after kinto error. Should request reconnect.
 implement a solution to unblock the user.
 display used quota. #837 

Fix #791 #801 #802 #827
@ryanfeeley
Copy link
Contributor

@vladikoff @sebastienbarbier Is that dialog copy + the Reconnect to Sync footer style appropriate?

@ryanfeeley ryanfeeley removed their assignment Apr 6, 2018
@cknowles-admin cknowles-admin added the ARCHIVED CLOSED at time of archiving label Jan 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ARCHIVED CLOSED at time of archiving
Projects
None yet
Development

No branches or pull requests

5 participants