quick-start icon indicating copy to clipboard operation
quick-start copied to clipboard

Avoid unnecessary API calls

Open samuelmeuli opened this issue 4 years ago • 0 comments

For simplicity, the current implementation calls the refreshBookmarks function to update the UI after every modification to the bookmark tree. This obviously isn't very efficient. Instead of calling this function, the flattened bookmark tree (in the Redux store) could be modified directly.

Example:

https://github.com/samuelmeuli/quick-start/blob/4daf1c457e67b9ee68d1ccd965c641e600ea5bba/src/content/store/thunks.ts#L24-L31

samuelmeuli avatar Apr 13 '20 11:04 samuelmeuli