-
Notifications
You must be signed in to change notification settings - Fork 407
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Remove bech32 #1385
feat: Remove bech32 #1385
Conversation
rodrigobranas
commented
Jun 20, 2024
- Closes Remove bech32 addresses completely #1309
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
---|---|---|---|
🟡 | Statements | 62.46% | 3223/5160 |
🔴 | Branches | 41.99% | 621/1479 |
🔴 | Functions | 47.11% | 684/1452 |
🟡 | Lines | 63% | 3106/4930 |
Test suite run success
243 tests passing in 71 suites.
Report generated by 🧪jest coverage report action from 2d3de68
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as the address is not clickable anymore (toggle behavior was removed), it doesnt need to be cursor: pointer anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
packages/app/src/systems/Network/components/NetworkSelector/NetworkSelector.test.tsx
Outdated
Show resolved
Hide resolved
packages/app/src/systems/Account/components/FuelAddress/FuelAddress.test.tsx
Outdated
Show resolved
Hide resolved
packages/app/src/systems/Account/components/FuelAddress/FuelAddress.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀 |
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated. # Releases ## [email protected] ### Minor Changes - [#1397](#1397) [`3966aa92`](3966aa9) Thanks [@rodrigobranas](https://github.com/rodrigobranas)! - Improved error handling and including location informations to support better understanding about the causes of erros - [#1385](#1385) [`d49b9efe`](d49b9ef) Thanks [@rodrigobranas](https://github.com/rodrigobranas)! - Support for Bech32 addresses has been removed and replaced with b256 (hex) addresses throughout the wallet screen components. All addresses, both existing and new, will now be displayed in the b256 format. While Bech32 addresses can still be used on the send screen, they will be automatically converted and processed as b256 internally. QR codes on the receive screen will now encode addresses in the b256 format. ### Patch Changes - Updated dependencies \[]: - @fuel-wallet/[email protected] ## @fuel-wallet/[email protected] ## @fuel-wallet/[email protected] Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>