Repository navigation
Release v3.13.0: Safe App receipts, GTM swap, copy fixes - #301
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Coverage Report for CI Build 39799741165Coverage increased (+0.05%) to 68.641%Details
Uncovered Changes
Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
tuler
approved these changes
Oct 22, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
That is a follow-up to the wallet migration (#300) before tagging v3.13.0. Inside a Safe App the transaction banner never confirmed, the last open dependency-audit finding (
react-gtm-module) is swapped and the transaction feedback copy is corrected. No user-facing changes beyond those.Changes:
Safe App receipts — the Safe provider answers
eth_sendTransactionwith the Safe transaction hash, not an on-chain hash, and only that provider can map it to a receipt once the multisig executes. After #300 the receipt was polled on the public HTTP client, which never finds that hash, so the banner spun until viem's 180 s timeout.useTransactionnow polls through the wallet client when the Safe connector is active.stake → Safe provider returns
safeTxHash→ owners sign and execute in Safe →walletClient.waitForTransactionReceipt→ provider mapssafeTxHashto the receipt → banner confirmedReceipt polling has no deadline again for every wallet (ethers'
wait()had none, viem defaults to 180 s), and the replaced-transaction check is skipped on the Safe path, where the provider only returns a placeholder transaction.Google Tag Manager —
react-gtm-module(abandoned 2020) →GoogleTagManagerfrom@next/third-parties, rendered byTrackerProviderin production only and only whenNEXT_PUBLIC_GOOGLE_TAG_MANAGER_IDis set. GA4 events go throughreact-ga4and are unchanged.Copy — "Withdrawed" → "Withdrawn", "Stake set" → "Staked", "a ENS" → "an ENS", "with success" → "successfully", and a capital letter after "hired!" / "created!".
@next/third-partiesreact-gtm-module,@types/react-gtm-moduleNotes
[3.13.0] - 2026-10-22added.