Automattic / bugomattic

Bugomattic is a tool that guides bug reporters to the right actions within large, complex organizations
GNU General Public License v2.0
6 stars 0 forks source link

Make all localstorage caches error safe #148

Closed dpasque closed 1 year ago

dpasque commented 1 year ago

What Does This PR Add/Change?

All our localstorage cache sets are now error safe, meaning a failure to cache will never disrupt site usage.

Now, we just leave a helpful console message that the user might be able to speed up the site by making some room in localstorage

Testing Instructions

This has to be tested on an internal deployment to use the real API -- I've taken care of that! I deployed on my sandbox and inflated local storage so there was no room, and verified all 4 keys don't throw errors and leave the console message.

Also, all unit tests are passing!

Issues

Related to #
Closes #147