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

Update SUI SDK from @mysten/sui.js to @mysten/sui #1797

Closed
wants to merge 10 commits into from

Conversation

karangoraniya
Copy link
Contributor

Description

This PR updates the project to use the latest SUI SDK, migrating from @mysten/sui.js to @mysten/sui. The change involves updating import statements and adjusting function calls to align with the new SDK version across multiple folders in the project.

Changes made:

  • Migrated SDK import from @mysten/sui.js to @mysten/sui
  • Updated import statements across the codebase
  • Refactored function calls to match the new SDK's API
  • Ensured compatibility with the latest SUI SDK version

Folders and files changed:

1. target_chains/sui/cli

cli.ts

  • Updated import statements
  • Modified function signatures to use SuiClient

pyth_deploy.ts

  • Updated import statements
  • Adjusted function signatures and return types

upgrade_pyth.ts

  • Updated import statements
  • Modified function signatures to use SuiClient

package.json

  • Updated to new @mysten/sui dependency version

2. target_chains/sui/sdk

examples/SuiRelay.ts

  • Updated import statements
  • Modified function signatures to use SuiClient

client.ts

  • Updated import statements
  • Modified function signatures to use SuiClient

package.json

  • Updated to new @mysten/sui dependency version

3. apps/price_pusher

command.ts

  • Updated import statements
  • Modified function signatures to use SuiClient

sui.ts

  • Updated import statements
  • Modified function signatures to use SuiClient

package.json

  • Updated to new @mysten/sui dependency version

Additional Notes

Please review the changes carefully, particularly in areas where SDK function calls have been modified.

Copy link

vercel bot commented Jul 25, 2024

@karangoraniya is attempting to deploy a commit to the pyth-web Team on Vercel.

A member of the Team first needs to authorize it.

@karangoraniya karangoraniya changed the title Update SUI SDK from @mysten/sui.js to @mysten/sui #2 Update SUI SDK from @mysten/sui.js to @mysten/sui Jul 25, 2024
@guibescos
Copy link
Contributor

gm gm, the formatting changes make it super hard to review... can you use prettier

@karangoraniya
Copy link
Contributor Author

gm gm, the formatting changes make it super hard to review... can you use prettier

gm! I am already using Prettier. Let me change the setting for single quotes
(') to double quotes (").

@ali-bahjati
Copy link
Collaborator

Thanks a lot, we've taken your work and added them in #1802

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