Persist transactions

[evgenyboxer]

Description

Added persistence to notifications by adding a version version prop to lsGet and lsSet.

Fixes # (issue)

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Other

How should this be tested?

Please provide instructions so we can test. Please also list any relevant details for your test configuration.

• Make a trade/invest, etc, and refresh the browser.

Checklist:

• I have performed a self-review of my own code
• I have commented my code where relevant, particularly in hard-to-understand areas
• My changes generate no new console warnings
• The base of this PR is master if hotfix, develop if not

[linear]

UI-682 Persist transactions

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

kovan-app – ./

🔍 Inspect: https://vercel.com/balancer/kovan-app/F1Pe7FCLxYWGMKFyUQCzwGZpLPAS
✅ Preview: Canceled

app – ./

🔍 Inspect: https://vercel.com/balancer/app/AHLRKEdJL4RWnzRHVkfo36mUbgXf
✅ Preview: Canceled

staging-app – ./

🔍 Inspect: https://vercel.com/balancer/staging-app/7xvVNKLVzMooZ3HmWugjXhhsGcpP
✅ Preview: https://staging-app-git-evgeny-ui-682-persist-transactions-balancer.vercel.app

staging-kovan-app – ./

🔍 Inspect: https://vercel.com/balancer/staging-kovan-app/BJ74MEoYoRKkMkvkfAgKboEoN38t
✅ Preview: https://staging-kovan-app-git-evgeny-ui-682-persist-tra-321eb4-balancer.vercel.app

gnosis – ./

🔍 Inspect: https://vercel.com/balancer/gnosis/CHp2fktxBsuR1BqN3bfbKsT6uop9
✅ Preview: https://gnosis-git-evgeny-ui-682-persist-transactions-balancer.vercel.app

polygon – ./

🔍 Inspect: https://vercel.com/balancer/polygon/5moLFt5iik1pyeQZcD2jJg9KzQ8y
✅ Preview: https://polygon-git-evgeny-ui-682-persist-transactions-balancer.vercel.app

beta-polygon – ./

🔍 Inspect: https://vercel.com/balancer/beta-polygon/FCr6EA2DNtZHEN1yM4hC5U4nKDiY
✅ Preview: https://beta-polygon-git-evgeny-ui-682-persist-transactions-balancer.vercel.app

[evgenyboxer]

Moved into lsGet and refactored (I'm not a fan of returning an empty object - better return something like null)

[evgenyboxer]

If provided with version it will try to get the data of that version and will always return the defaultValue in case of a mismatch (for version upgrades)

[garethfuller]

I think the local storage logic could probably be extracted into a service now. Would make it cleaner and more self-contained. I'm not suggesting it needs to be part of this PR though.

[evgenyboxer]

Yeah, good idea - a chance to make it much nicer and cleaner.