-
Notifications
You must be signed in to change notification settings - Fork 165
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.
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
#3640 - macro after reloading the page monomers added to the favorites section disappear #3688
#3640 - macro after reloading the page monomers added to the favorites section disappear #3688
Conversation
73b4283
to
dd47951
Compare
state.favorites = localFavorites; | ||
}, | ||
|
||
unsetFavoriteMonomersFromLocalStorage: (state: RootState) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would propose to rename it to clearFavoritesState() because this method does not unset something from local storage
}); | ||
}, | ||
|
||
unsetFavoritePresetsFromLocalStorage: (state: RootState) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.
setItem: (key: string, item: unknown) => void; | ||
} | ||
|
||
class LocalStorage implements LocalStorageCopy { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would propose to rename it to LocalStorageWrapper because LocalStorage can be read as native localStorage.
Also why we need interface here? Do we plan to have other classes which implement it?
How the feature works? / How did you fix the issue?
(Screenshots, videos, or GIFs, if applicable)
closes #3640
Check list
#1234 – issue name