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

[ts-sdk] tx block rename #9941

Merged
merged 1 commit into from
Mar 27, 2023
Merged

[ts-sdk] tx block rename #9941

merged 1 commit into from
Mar 27, 2023

Conversation

Jordan-Mysten
Copy link
Contributor

@Jordan-Mysten Jordan-Mysten commented Mar 27, 2023

Description

Updating naming of transaction builder APIs.

Test Plan

Tests should still pass.

@vercel
Copy link

vercel bot commented Mar 27, 2023

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

Name Status Preview Comments Updated
explorer ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 27, 2023 at 6:38PM (UTC)
explorer-storybook ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 27, 2023 at 6:38PM (UTC)
wallet-adapter ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 27, 2023 at 6:38PM (UTC)

Copy link
Contributor Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 27, 2023 18:12 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 27, 2023 18:12 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Mar 27, 2023

⚠️ 🦋 Changesets Warning: This PR has changes to public npm packages, but does not contain a changeset. You can create a changeset easily by running pnpm changeset in the root of the Sui repo, and following the prompts. If your change does not need a changeset (e.g. a documentation-only change), you can ignore this message. This warning will be removed when a changeset is added to this pull request.

Learn more about Changesets.

@Jordan-Mysten Jordan-Mysten force-pushed the jordan--SDK_rename_methods branch from e508573 to 069cfd2 Compare March 27, 2023 18:21
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 27, 2023 18:22 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 27, 2023 18:22 Inactive
@Jordan-Mysten Jordan-Mysten force-pushed the jordan--SDK_rename_methods branch from 069cfd2 to ef52e9c Compare March 27, 2023 18:34
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 27, 2023 18:35 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 27, 2023 18:36 Inactive
@Jordan-Mysten Jordan-Mysten force-pushed the jordan--SDK_rename_methods branch from ef52e9c to ca08511 Compare March 27, 2023 18:37
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 27, 2023 18:37 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 27, 2023 18:38 Inactive
@Jordan-Mysten Jordan-Mysten merged commit 6cba50d into main Mar 27, 2023
@Jordan-Mysten Jordan-Mysten deleted the jordan--SDK_rename_methods branch March 27, 2023 19:11
@@ -64,7 +64,7 @@ function TransferCoinPage() {
});

return signer.signAndExecuteTransaction({
Copy link
Contributor

Choose a reason for hiding this comment

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

should we also rename this to signAndExecuteTransactionBlock?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah happening in this follow-up: #9946

Jordan-Mysten added a commit that referenced this pull request Mar 27, 2023
## Description 

This is a follow-up for #9941, renaming provider methods and wallet
adapter methods.

## Test Plan 

Tests should still pass.
666lcz pushed a commit that referenced this pull request Mar 28, 2023
## Description 

Updating naming of transaction builder APIs.

## Test Plan 

Tests should still pass.
666lcz pushed a commit that referenced this pull request Mar 28, 2023
This is a follow-up for #9941, renaming provider methods and wallet
adapter methods.

Tests should still pass.
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.

3 participants