5afe / safe-react

Deprecated! New repo – https://github.com/safe-global/web-core
MIT License
332 stars 363 forks source link

Release/3.31.1 #4082

Closed usame-algan closed 2 years ago

usame-algan commented 2 years ago

Changes

github-actions[bot] commented 2 years ago

ESLint Summary View Full Report

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 1 N/A

Report generated by eslint-plus-action

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 3143637288


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/AppLayout/Header/components/SafeTokenWidget/index.tsx 0 5 0.0%
src/routes/safe/components/Balances/SendModal/screens/SendFunds/index.tsx 0 5 0.0%
src/components/AppLayout/Header/components/SafeTokenWidget/useSafeTokenAllocation.ts 0 22 0.0%
<!-- Total: 2 34 5.88% -->
Totals Coverage Status
Change from base Build 3136946051: -0.1%
Covered Lines: 4776
Relevant Lines: 10389

💛 - Coveralls
github-actions[bot] commented 2 years ago

Deployment links

:white_circle: Mainnet     :purple_circle: Polygon     :orange_circle: Rinkeby

francovenica commented 2 years ago

Ok, Looks good. I was able to confirm the non-transferable nature of the safe token, and Jago checked the EIP712domain thing. The claim app doesn't show in this PR, I assume is because of the enviroment, so I cannot be sure about the "Display total allocation"

The rest is also fine: safe creation, signing, send funds, contract interaction, address book, editing owners and policies. Signing with MM, WC MM phone app, trezor and ledger