Skip to content

Port transaction hash bug to v3#6429

Merged
zoeyTM merged 4 commits intov-nextfrom
fix/tx-hash-bug-v3
May 5, 2025
Merged

Port transaction hash bug to v3#6429
zoeyTM merged 4 commits intov-nextfrom
fix/tx-hash-bug-v3

Conversation

@zoeyTM
Copy link
Contributor

@zoeyTM zoeyTM commented Mar 3, 2025

Ported this bug fix from Ignition in Hardhat 2 to Hardhat 3.

@changeset-bot
Copy link

changeset-bot bot commented Mar 3, 2025

🦋 Changeset detected

Latest commit: ee8bcbe

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@nomicfoundation/ignition-core Patch
@nomicfoundation/ignition-ui Patch
@nomicfoundation/hardhat-ignition Patch
hardhat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@zoeyTM zoeyTM requested a review from kanej March 3, 2025 09:13
@vercel
Copy link

vercel bot commented Mar 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
hardhat ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 12, 2025 3:30am

@fvictorio fvictorio added this to Hardhat Mar 3, 2025
@github-project-automation github-project-automation bot moved this to Backlog in Hardhat Mar 3, 2025
@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch from 3b1227d to 7275da6 Compare March 3, 2025 09:14
@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch from 517dbe0 to e75f03d Compare March 3, 2025 09:49
@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch from e75f03d to 8bad5d4 Compare March 12, 2025 03:30
@kanej kanej linked an issue Mar 18, 2025 that may be closed by this pull request
@schaable schaable added the v-next A Hardhat v3 development task label Apr 10, 2025
@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch 3 times, most recently from 03b6c6f to 469f251 Compare April 29, 2025 08:08
Copy link
Member

@kanej kanej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have been through and done a comparison with the v2 PR. I left a few questions, but this is looking good.

Can I ask how I test this manually?

@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch 3 times, most recently from dc81860 to 3f5f5b0 Compare May 5, 2025 08:52
Copy link
Member

@kanej kanej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have done a file by file comparison and it looks good. Lets get this merged.

@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch from 3f5f5b0 to ca3b5ce Compare May 5, 2025 09:32
@zoeyTM zoeyTM force-pushed the fix/tx-hash-bug-v3 branch from ca3b5ce to ee8bcbe Compare May 5, 2025 09:39
@zoeyTM zoeyTM enabled auto-merge May 5, 2025 09:41
@zoeyTM zoeyTM added this pull request to the merge queue May 5, 2025
Merged via the queue into v-next with commit 8d16859 May 5, 2025
114 checks passed
@zoeyTM zoeyTM deleted the fix/tx-hash-bug-v3 branch May 5, 2025 09:48
@github-project-automation github-project-automation bot moved this from Backlog to Done in Hardhat May 5, 2025
@zoeyTM zoeyTM mentioned this pull request May 7, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status:triaging v-next A Hardhat v3 development task

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Port Ignition hash bug fix

3 participants