Follow-up from "Add a button to export CSV data": persist storage to DB
The following discussion from !26838 (merged) should be addressed:
-
@markrian started a discussion: (+2 comments) Not blocking: While persisting to local storage is fine, especially for an MVC, the UX could be improved by instead persisting to the database via
user_callouts_helper. This way, the state of the popover will persist across all browsers (or the same browser after alocalStorage.clear()) for the user.See !22117 (merged) for an example of how to do that. There's also a video which goes through another implementation: https://www.youtube.com/watch?v=wJmI4tA92L8.
That said, it's a fair bit more code for a marginal UX improvement, especially when you have to pass through a handful of values from the backend to a descendant component
🤷 With all that said, assuming you'll keep this
localStorageimplementation, I'd recommend that you open a follow-up issue to track clearing up this data for when the popover gets removed.Thoughts?