Skip to content

fix: Wait for receipt #8358

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.

Sign up for GitHub

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

Merged
PhilWindle merged 6 commits into from
Sep 5, 2024
Merged

fix: Wait for receipt #8358

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
491cc6c
Wait for receipt
PhilWindle Sep 3, 2024
9f65e3c
Update yarn-project/ethereum/src/deploy_l1_contracts.ts
PhilWindle Sep 4, 2024
ef16fd1
Merge branch 'master' into pw/wait-for-tx
PhilWindle Sep 4, 2024
34a7658
Merge branch 'master' into pw/wait-for-tx
PhilWindle Sep 4, 2024
b0d51f2
Merge branch 'master' into pw/wait-for-tx
PhilWindle Sep 5, 2024
190b0bf
Merge branch 'master' into pw/wait-for-tx
PhilWindle Sep 5, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion yarn-project/ethereum/src/deploy_l1_contracts.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,10 @@ export const deployL1Contracts = async (
// fund the portal contract with Fee Juice
const FEE_JUICE_INITIAL_MINT = 20000000000;
const mintTxHash = await feeJuice.write.mint([feeJuicePortalAddress.toString(), FEE_JUICE_INITIAL_MINT], {} as any);
txHashes.push(mintTxHash);

// @note This is used to ensure we fully wait for the transaction when running against a real chain
// otherwise we execute subsequent transactions too soon
await publicClient.waitForTransactionReceipt({ hash: mintTxHash });
logger.info(`Funding fee juice portal contract with fee juice in ${mintTxHash}`);

if ((await feeJuicePortal.read.registry([])) === zeroAddress) {
Expand Down