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

feat: Agave v2 RPC: replace getConfirmedTransaction with getTransaction #3418

Conversation

buffalojoec
Copy link
Contributor

@buffalojoec buffalojoec commented Oct 23, 2024

Problem

With the upcoming upgrade from 1.18 to 2.0 on Solana mainnet-beta, deprecated RPC methods have been removed, therefore they will no longer be available through Web3.js client requests.

The Agave 2.0 Migration Guide lists semi-equivalent RPC method counterparts for each of the removed methods.

Summary of Changes

Replace getConfirmedTransaction with getTransaction. This method was already requiring a confirmation level of confirmed or finalized, so the behavior should be unchanged.

Copy link

changeset-bot bot commented Oct 23, 2024

⚠️ No Changeset found

Latest commit: 3065292

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ branch from 2fe2282 to 6acd1dc Compare October 29, 2024 07:05
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from 950dba5 to ee67085 Compare October 29, 2024 07:05
@buffalojoec buffalojoec marked this pull request as ready for review October 29, 2024 07:05
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ branch from 6acd1dc to 22a65a5 Compare October 29, 2024 09:54
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from ee67085 to f96e4d2 Compare October 29, 2024 09:54
lorisleiva
lorisleiva previously approved these changes Oct 29, 2024
mcintyre94
mcintyre94 previously approved these changes Oct 29, 2024
@github-actions github-actions bot added stale and removed stale labels Nov 13, 2024
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ branch from 22a65a5 to 77b6e59 Compare November 18, 2024 11:49
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from f96e4d2 to 1dbecf8 Compare November 18, 2024 11:49
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ branch from 77b6e59 to c35251a Compare November 27, 2024 11:56
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from 1dbecf8 to ac08a16 Compare November 27, 2024 11:56
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ branch from c35251a to 6607d91 Compare December 6, 2024 07:26
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from ac08a16 to e23d27c Compare December 6, 2024 07:26
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ branch from 6607d91 to e730045 Compare December 6, 2024 07:29
@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from e23d27c to e71da6f Compare December 6, 2024 07:29
steveluscher
steveluscher previously approved these changes Dec 15, 2024
Copy link
Contributor Author

buffalojoec commented Dec 16, 2024

Merge activity

  • Dec 15, 11:14 PM EST: A user started a stack merge that includes this pull request via Graphite.
  • Dec 15, 11:17 PM EST: Graphite rebased this pull request as part of a merge.
  • Dec 15, 11:18 PM EST: Graphite couldn't merge this PR because it was not satisfying all requirements (PR does not have required approvals).

@buffalojoec buffalojoec changed the base branch from 10-23-feat_agave_v2_rpc_replace_getconfirmedblock_with_getblock_ to graphite-base/3418 December 16, 2024 04:15
@buffalojoec buffalojoec changed the base branch from graphite-base/3418 to maintenance/v1.x December 16, 2024 04:15
@buffalojoec buffalojoec dismissed steveluscher’s stale review December 16, 2024 04:15

The base branch was changed.

@buffalojoec buffalojoec dismissed stale reviews from mcintyre94 and lorisleiva December 16, 2024 04:15

The base branch was changed.

@buffalojoec buffalojoec force-pushed the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch from e71da6f to 3065292 Compare December 16, 2024 04:16
Copy link
Contributor

A preview of the GitHub Pages site based on this PR is now available here:

solana-labs.github.io/solana-web3.js-pr-preview/3418/

@buffalojoec buffalojoec merged commit a805cb9 into maintenance/v1.x Dec 16, 2024
11 checks passed
@buffalojoec buffalojoec deleted the 10-23-feat_agave_v2_rpc_replace_getconfirmedtransaction_with_gettransaction_ branch December 16, 2024 04:27
Copy link
Contributor

🎉 This PR is included in version 1.97.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants