-
Notifications
You must be signed in to change notification settings - Fork 39
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
docs: drop all references to reallocate() and its helpers #1266
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved presuming you'll make the requested changes
contract-->>-zoe: reallocate() | ||
contract-->>-zoe: atomicRearrange() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the corresponding .svg should be regenerated.
it shows up in Specifying Offers
https://10871-excisereallocate.documentation-7tp.pages.dev/assets/zoe-simp.1Ml6HzzR.svg
Crud: CONTRIBUTING.md doesn't say how. IOU, I suppose.
In brief: I use Mermaid Editor - Visual Studio Marketplace.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That page also has a link to a web editor which works fine for small diagrams like this, and allowed me to save as svg.
main/guides/zoe/index.md
Outdated
@@ -21,7 +21,7 @@ Trading with a contract using Zoe typically goes through these steps: | |||
1. Parties call [E(zoe).offer(invitation, proposal, assets)](/reference/zoe-api/zoe#e-zoe-offer-invitation-proposal-paymentpkeywordrecord-offerargs). Zoe escrows the assets. | |||
2. Zoe relays the proposal to the contract identified in the invitation. | |||
3. The contract handles proposals using its custom business logic. | |||
4. The contract instructs Zoe to [reallocate](/reference/zoe-api/zoe-contract-facet#zcf-reallocate-seats) assets among the parties. | |||
4. The contract instructs Zoe to [reallocate](/reference/zoe-api/zoe-contract-facet.html#zcf-atomicrearrange-transfers) assets among the parties. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4. The contract instructs Zoe to [reallocate](/reference/zoe-api/zoe-contract-facet.html#zcf-atomicrearrange-transfers) assets among the parties. | |
4. The contract instructs Zoe to [reallocate](/reference/zoe-api/zoe-contract-facet#zcf-atomicrearrange-transfers) assets among the parties. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Deploying documentation with Cloudflare Pages
|
Cloudflare deployment logs are available here |
closes: #798
See #10871
We converted from
zcf.reallocate()
tozcf.atomicRearrange()
over the course of several releases. The old code will be completely gone as of #10871. This PR cleans up the last of the remaining documentation.