Skip to content

Conversation

dan-ziv
Copy link

@dan-ziv dan-ziv commented May 2, 2022

Description of the Changes

Issue: when the balance is zero with more than 5 digits after the dot (i.e. 0.0000001) balance is shown as 0.
Solution: transform the displayed balances to strings and format their display to 0.00000... in such cases.

Checklist

  • Changes have been done against master branch, and PR does not conflict
  • New unit / functional tests have been added (whenever applicable)
  • Test are passing in local environment
  • Docs have been updated
  • PR title is follow the Conventional Commits convention: <type>[optional scope]: <description>, e.g: fix: prevent racing of requests

This change is Reviewable

@dan-ziv dan-ziv merged commit 2906e3e into dev May 3, 2022
@dan-ziv dan-ziv deleted the fix/cancel-to-fixed-in-balances branch May 3, 2022 08:34
dan-ziv pushed a commit that referenced this pull request May 3, 2022
 ### [1.3.1-dev.2](v1.3.1-dev.1...v1.3.1-dev.2) (2022-05-03)

### 🔧 Fixes

* cancel toFixed when format balances ([#172](#172)) ([2906e3e](2906e3e))
@dan-ziv
Copy link
Author

dan-ziv commented May 3, 2022

🎉 This PR is included in version 1.3.1-dev.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

dan-ziv pushed a commit that referenced this pull request May 4, 2022
 ## [1.4.0](v1.3.0...v1.4.0) (2022-05-04)

### 🔧 Fixes

* cancel toFixed when format balances ([#172](#172)) ([2906e3e](2906e3e))
* connection modal overrides gsw selection modal ([#175](#175)) ([73b1f21](73b1f21))
* temporary disable max button ([#173](#173)) ([1d3038d](1d3038d))
* update app state on accountsChanged event ([#176](#176)) ([3161cc9](3161cc9))
* update text for the attempt to deposit more than the allowable amount ([#174](#174)) ([d8f3da8](d8f3da8))

### 🧩 Features

* add asset to SN wallet after a successful deposit ([#171](#171)) ([30d8ec3](30d8ec3))
@dan-ziv
Copy link
Author

dan-ziv commented May 4, 2022

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@dan-ziv dan-ziv restored the fix/cancel-to-fixed-in-balances branch September 6, 2022 11:18
@dan-ziv dan-ziv deleted the fix/cancel-to-fixed-in-balances branch September 6, 2022 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant