sui: changes for Release v1.5.1 - #220
Merged
Merged
Conversation
* feat: add collections for sui * chore: Add cla assistant workflow (#213) * chore: Add cla assistant workflow * fix: Templatize repo name with case modification * sui: Add Allowance and Vesting Wallet (#209) * add allowance package docs (spend_vault) correct allowance docs: single publish flow, address-balance delivery, working checkpoints * add VestingWallet docs (#202) * add VestingWallet docs * ref: fix merge conflict resolution Co-authored-by: Nenad <xinef.it@gmail.com> * Update llms.txt with Allowance and Finance links Added links for Allowance and Finance packages in the documentation. Add link to 'Learn — Delegated Spending' guide Added a link to the 'Learn — Delegated Spending' guide in the Sui Contracts section. * docs: expand delegated spending tutorial setup and signer switches --------- Co-authored-by: kosedogus <dogus.kose@openzeppelin.com> * Update CONTRIBUTING.md (#216) add CLA blurb --------- Co-authored-by: Sai Tirumerla <57160285+tirumerla@users.noreply.github.com> Co-authored-by: Nenad <nenad.misic@openzeppelin.com> Co-authored-by: kosedogus <dogus.kose@openzeppelin.com> Co-authored-by: johndawsonneufeld <johndawsonneufeld@gmail.com>
* feat: add timelock docs * chore: Add cla assistant workflow (#213) * chore: Add cla assistant workflow * fix: Templatize repo name with case modification * sui: Add Allowance and Vesting Wallet (#209) * add allowance package docs (spend_vault) correct allowance docs: single publish flow, address-balance delivery, working checkpoints * add VestingWallet docs (#202) * add VestingWallet docs * ref: fix merge conflict resolution Co-authored-by: Nenad <xinef.it@gmail.com> * Update llms.txt with Allowance and Finance links Added links for Allowance and Finance packages in the documentation. Add link to 'Learn — Delegated Spending' guide Added a link to the 'Learn — Delegated Spending' guide in the Sui Contracts section. * docs: expand delegated spending tutorial setup and signer switches --------- Co-authored-by: kosedogus <dogus.kose@openzeppelin.com> * Update CONTRIBUTING.md (#216) add CLA blurb --------- Co-authored-by: Sai Tirumerla <57160285+tirumerla@users.noreply.github.com> Co-authored-by: Nenad <nenad.misic@openzeppelin.com> Co-authored-by: kosedogus <dogus.kose@openzeppelin.com> Co-authored-by: johndawsonneufeld <johndawsonneufeld@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Add Token Sale docs * Update llms.txt with new documentation links Added links for PrefundedSale and Sale in the documentation. * Add overview link for Sale package in llms.txt * update token sale API
* merge conflicts * update remaining versions in links
✅ Deploy Preview for openzeppelin-docs-v2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
0xNeshi
changed the title
Removes a duplicated Finance folder block in the Sui sidebar nav and aligns the new Collections/Timelock/Sale pages with the "onchain" (one word) spelling used elsewhere in the docs.
stevep0z
approved these changes
Jul 29, 2026
stevep0z
left a comment
stevep0z
left a comment
Collaborator
There was a problem hiding this comment.
Thanks for this @0xNeshi and team! Looks good to me. I made a commit for:
- Removed duplicate Finance nav entry
- Applying the typical stylistic reformatting check we do for docs (nit for onchain vs on-chain)
Something to note for future feature pages within the docs, within some package pages, like fixed-point-math, we have more detailed explanations and design notes, whereas others we have less (collections for example). This is fine for now, though it would be good to lean towards more design notes for developers and LLMs.
Once this is merged, we will update the docs with PR #222
Contributor
Author
|
quote from #220 (review):
cc @ericnordelo |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Documentation Pull Request
Summary
Type of Change
Related Issues
Fixes #
Relates to #
Checklist
pnpm run buildpnpm run checkAdditional Notes