sui
sui copied to clipboard
Update README.md
update method used in examples
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 | Feb 6, 2023 at 7:51PM (UTC) |
frenemies | ✅ Ready (Inspect) | Visit Preview | 💬 Add your feedback | Feb 6, 2023 at 7:51PM (UTC) |
wallet-adapter | ✅ Ready (Inspect) | Visit Preview | 💬 Add your feedback | Feb 6, 2023 at 7:51PM (UTC) |
1 Ignored Deployment
Name | Status | Preview | Comments | Updated |
---|---|---|---|---|
explorer-storybook | ⬜️ Ignored (Inspect) | Feb 6, 2023 at 7:51PM (UTC) |
the getOwnedObjectRefs method does not exist in the JsonRpcProvider class, i changed it to the correct method getObjectRef
the getOwnedObjectRefs method does not exist in the JsonRpcProvider class, i changed it to the correct method getObjectRef
Thank you!
⚠️ 🦋 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.
getOwnedObjectRefs
was replaced by getObjectsOwnedByAddress
here
Using getObjectRef is not correct here since the example is about fetching objects owned by an address and getObjectRef
fetches the reference of an object.
It seems though readme already has been updated on main.
cc @kchalkias
pchrysochoidis is right, i think that can close this request