-
Notifications
You must be signed in to change notification settings - Fork 411
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
Release 1.37.1 #3766
Release 1.37.1 #3766
Conversation
* fix: Hide signed transaction from untrusted pending queue * fix: Revert label adjustment, extract logic from hook and write tests * refactor: Filter items first
…posal (#3712) * fix: show loading state in WC widget between pairing and receiving a proposal * fix: clear loading state on failed paring * empty commit for deployment * disable loading state for approval form * fix typo * add 30 second timeout for proposal * Only show timeout error when waiting on proposal
* Fix: update social login disclaimer * Prettier
* Fix: memoize Safe List data * Adjust types
* tests: add component tests for tx history filter * tests: update tests
Bumps [next](https://github.com/vercel/next.js) from 14.1.0 to 14.1.1. - [Release notes](https://github.com/vercel/next.js/releases) - [Changelog](https://github.com/vercel/next.js/blob/canary/release.js) - [Commits](vercel/next.js@v14.1.0...v14.1.1) --- updated-dependencies: - dependency-name: next dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Branch preview⏳ Deploying a preview site... |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
📦 Next.js Bundle Analysis for safe-wallet-webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success1436 tests passing in 197 suites. Report generated by 🧪jest coverage report action from 7537a62 |
* feat: add new label to swaps button
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Question regarding the infura token for dev. |
I tried to check that las commit regarding the change in how tx in bulk is tracked in analytics, but I don't see any GTM calls all of the sudden. Those were there before since I even reported an issue about them in the board. |
@francovenica yes, exactly, the RC branches will use all the production settings. CGW will also be the prod version. |
It seems that the GTM calls are not being shown anymore because now the release branch is "like prod". I went back to the PR's that did the changes in the GTM tracks 3779 and 3783 and test those fixed there and they work fine. |
The rest of the regression went fine, and the last commits were checked and they worked fine. There is an ongoing discussion if we should keep the "use different infura for dev" change, so if it is decied to roll back that change let me know If not, then the RC looks good for release |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
Features
Analytics
Bug fixes
WalletConnect
Fix: back button style (Fix: back button style #3742)
Fix: missing feature toggles (Fix: missing feature toggles #3748)
fix: do not show nonce conflict warning for batch transactions. (fix: do not show nonce conflict warning for batch transactions. #3608)
Fix: rearrange Settings tabs (Fix: rearrange Settings tabs #3751)
Fix: memoize Safe List data (Fix: memoize Safe List data #3729)
Fix: move rhino.fi to warned bridges (Fix: move rhino.fi to warned bridges #3740)
fix: Hide signed transaction from untrusted pending queue (fix: Hide signed transaction from untrusted pending queue #3710)
fix: Add network to Rabby if unknown (fix: Add network to Rabby if unknown #3776)
Chores & tests