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 wallet standard docs #21447

Open
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

karangoraniya
Copy link
Contributor

Description

Updated import names in documentation to use the Standard* prefix in accordance with the latest changes in @mysten/wallet-standard.

The following updates were made:

  • ConnectFeatureStandardConnectFeature
  • ConnectMethodStandardConnectMethod
  • EventsFeatureStandardEventsFeature
  • EventsOnMethodStandardEventsOnMethod

This is a documentation update only, with no functional changes.

Test plan

No testing required as this is a documentation update.

Release notes

No release notes required as this change only affects documentation.

  • Protocol:
  • Nodes (Validators and Full nodes):
  • gRPC:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:

karangoraniya and others added 28 commits June 9, 2024 20:20
Copy link

vercel bot commented Mar 8, 2025

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

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 8, 2025 5:28pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Mar 8, 2025 5:28pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Mar 8, 2025 5:28pm

@github-actions github-actions bot added the Type: Documentation Improvements or additions to documentation label Mar 8, 2025
@hayes-mysten
Copy link
Contributor

This change looks good, thanks for updating these! Can we clean up the commit history so that we just have a single commit before merging?

@karangoraniya
Copy link
Contributor Author

This change looks good, thanks for updating these! Can we clean up the commit history so that we just have a single commit before merging?

Thank you! Yeah, sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants