Closed katspaugh closed 1 week ago
β Deploy successful!
Website: https://amount--walletweb.review.5afe.dev/home?safe=eth:0xA77DE01e157f9f57C7c4A326eeE9C4874D0598b6
Storybook: https://amount--walletweb.review.5afe.dev/storybook/
Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.
Type | Occurrences | Fixable |
---|---|---|
Errors | 0 | 0 |
Warnings | 0 | 0 |
Ignored | 0 | N/A |
Report generated by eslint-plus-action
This analysis was generated by the Next.js Bundle Analysis action. π€
Page | Size (compressed) |
---|---|
global |
1003.77Β KB (π‘ +24Β B) |
The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.
Any third party scripts you have added directly to your app using the <script>
tag are not accounted for in this analysis
If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
π‘ | Statements | 79.16% (-0.01% π») |
11554/14595 |
π΄ | Branches | 58.33% (-0.02% π») |
2794/4790 |
π‘ | Functions | 66.12% (-0.04% π») |
1850/2798 |
π’ | Lines | 80.49% (-0.02% π») |
10415/12940 |
1444 tests passing in 199 suites.
Report generated by π§ͺjest coverage report action from 6c09d844d2625a25564d05fa6f4a56a0c3c83b8d
Verified
What it solves
Resolves https://github.com/5afe/safe-support/issues/464
A user reported a regression after #3807. Token amounts should not be abbreviated in tx details.
How this PR fixes it
It makes an exception for token amounts displayed in tx details to show the full number up to 30 digits. If more than 30, it will be still abbreviated.
After the fix: