Skip to content
This repository has been archived by the owner on Mar 16, 2023. It is now read-only.

Fix issues in sdk #469

Merged
merged 29 commits into from
Feb 13, 2020
Merged

Fix issues in sdk #469

merged 29 commits into from
Feb 13, 2020

Conversation

LeilaWang
Copy link
Collaborator

Summary

This PR fixes multiple minor issues in the sdk

  • show aztec address instead of real address in register view
  • return big numbers from all linked token related apis (Closes balanceOfLinkedToken returns an overflown number #468 )
  • correctly format large values in the ui
  • make subscriptions and callbacks more reliable while switching accounts
  • remove dynamic data like apiKey or network id from local storage to avoid conflicts between different dapps

This PR also introduces a mechanism to modify indexedDB and local storage if necessary in the future versions of the sdk.

Testing instructions

Tests for indexedDB migrations are in packages/extension/src/background/tasks/__tests__/migrateIndexedDB.spec.js

Tests for upgradable local storage data are integrated into existing tests in packages/extension/src/background/services/NoteService/__tests__/

Types of changes

Bug fix (non-breaking change which fixes an issue)

New feature (non-breaking change which adds functionality)

@ArnSch
Copy link
Contributor

ArnSch commented Feb 10, 2020

@LeilaWang just pinging about lint issue when you have a second.

@joeandrews joeandrews merged commit 30c0375 into develop Feb 13, 2020
@joeandrews
Copy link
Contributor

🎉 This PR is included in version 1.19.0 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 1.5.0 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 1.7.0 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 0.17.0 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 1.18.0 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 2.4.0 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 1.3.2 🎉

Your semantic-release bot 📦🚀

@joeandrews
Copy link
Contributor

🎉 This PR is included in version 1.1.4 🎉

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

balanceOfLinkedToken returns an overflown number
3 participants