-
Notifications
You must be signed in to change notification settings - Fork 206
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
refactor: bring autoswap up to date on the new Zoe spike branch #1411
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.
Looks good! I need to learn more about how addEmptySeat
works - it was surprising to me that it was asynchronous.
throw new Error('getPoolKeyword: brand not found'); | ||
}; | ||
|
||
const { zcfSeat: poolSeat } = await zcf.addEmptySeat(); |
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.
is this meant to be async? If so, why not use makeEmptyOffer?
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.
Dropped the async
. Thanks.
}); | ||
|
||
return { publicFacet, creatorFacet: publicFacet }; |
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.
Ooo nice, I like the concept of passing the publicFacet back as the creatorFacet. We might consider always passing the publicFacet back to the creator too.
addresses #1267, for autoswap. I think that bug also refers to multipoolAutoswap, so I won't close it. |
Based on the synchronous seat work. The unit tests for autoswap pass. I haven't yet done the swingset tests.
No, it was my oversight. |
1702c1c
to
1016d35
Compare
This brings the autoswap contract up-to-date on the new-zoe-spike-2 branch, based on the synchronous seat work.
The unit tests for autoswap pass. I haven't yet done the swingset tests.