Skip to content
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

accounts/abi/bind: change to use hash as parameter #30079

Merged
merged 3 commits into from
Dec 17, 2024

Conversation

mask-pp
Copy link
Contributor

@mask-pp mask-pp commented Jun 27, 2024

Users prefer to use a hash instead of a tx point.

@MariusVanDerWijden
Copy link
Member

Oh I like this change a lot. The problem is that this changes a much used API. I would be fine with breaking it, but I know @fjl doesn't like us breaking downstream users.

@mask-pp
Copy link
Contributor Author

mask-pp commented Jun 27, 2024

Oh I like this change a lot. The problem is that this changes a much used API. I would be fine with breaking it, but I know @fjl doesn't like us breaking downstream users.

In fact, users prefer to use hash. Users all know this function, but because they have to pass in tx, they have to choose to implement this function themselves.

@mask-pp
Copy link
Contributor Author

mask-pp commented Jun 27, 2024

Oh I like this change a lot. The problem is that this changes a much used API. I would be fine with breaking it, but I know @fjl doesn't like us breaking downstream users.

I often make wheels like this repeatedly. I think this change is very necessary for users. Thanks for your understanding!

@karalabe
Copy link
Member

karalabe commented Jul 2, 2024

So, since we ourselves use this in all abigen stuff, we can't break the API, sorry. But we can introduce a new method for waiting on a hash (or deploying a hash).

@MariusVanDerWijden
Copy link
Member

@mask-pp I pushed a commit on top that adds WaitMinedHash and WaitDeployedHash.
This is the only way we can change this api.
Please check it it looks good to you

@mask-pp
Copy link
Contributor Author

mask-pp commented Jul 21, 2024

@mask-pp I pushed a commit on top that adds WaitMinedHash and WaitDeployedHash. This is the only way we can change this api. Please check it it looks good to you

It's great, thanks for your commit.

@holiman holiman added this to the 1.14.13 milestone Dec 17, 2024
@holiman holiman merged commit 3c208cd into ethereum:master Dec 17, 2024
3 checks passed
gzliudan pushed a commit to gzliudan/XDPoSChain that referenced this pull request Jan 11, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 14, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 16, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 17, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 22, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 22, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 23, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
@fjl fjl modified the milestones: 1.14.13, 1.15.0 Jan 23, 2025
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Jan 24, 2025
This change adds methods which makes it possible for to wait for a transaction with a specific hash when deploying contracts during abi bind interaction.

---------

Co-authored-by: Marius van der Wijden <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants