search

aecreations / readnext

Read Next: an extension for Mozilla Firefox
Other
4 stars 1 forks source link

Duplicated reading list items #70

Open aecreations opened 1 year ago

aecreations commented 1 year ago

Read Next 1.0.1

Sporadic issue with duplicated reading list items. This was observed once, and it seems that there are two conditions that trigger it: 1) it is taking longer to load; and, 2) as soon as the reading list data is loaded from local storage (fallback action if network connection times out), the sync successfully finishes and the data retrieved from Dropbox is appended to the data loaded from local storage.

Suggested fix: When sync is finished, clear the reading list before populating it with the data retrieved from Dropbox.

aecreations commented 1 year ago

The reading list is already being cleared when it is built. See sidebar.js, function rebuildReadingList().

It's possible the sidebar is receiving extension messages reload-bookmarks-event (when sync finishes successfully) and sync-failed-netwk-error at the same time. There isn't a way to detect the two events happening at around the same time and cancelling one of the two.