Closed schmanu closed 3 weeks ago
β Deploy successful!
Storybook: https://fix_recovery_detection--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
I guess it's a regression after #3811. So we cannot just always use the latest multisend contract?
This analysis was generated by the Next.js Bundle Analysis action. π€
Page | Size (compressed) |
---|---|
global |
948.78Β KB (-3Β 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.44% (+0% πΌ) |
11515/14496 |
π΄ | Branches | 58.45% (+0.01% πΌ) |
2784/4763 |
π‘ | Functions | 66.74% | 1846/2766 |
π’ | Lines | 80.76% (+0% πΌ) |
10381/12854 |
1436 tests passing in 199 suites.
Report generated by π§ͺjest coverage report action from 23cdfa01c6fbdd6d0213f87990a4566a578a072b
So we cannot just always use the latest multisend contract?
We use the version that we pass to the safe-core-sdk. But if that version has no MultiSendCallOnly contract it falls back to 1.3.0. Maybe we could use the getMultiSendCallOnlyContract function of the safe-core-sdk instead of using safe-deployments. Then it would always be aligned.
Closing this as we reverted the change that caused this issue.
What it solves
Resolves https://www.notion.so/safe-global/Recovery-tx-is-being-considered-a-malicious-tx-724664e7de4e4a8da4d88b5606af047c?pvs=4
How this PR fixes it
How to test it
Checklist