-
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
feat(orchestrator): return unwrapped vows #9537
Conversation
ae9ecf5
to
b3802fc
Compare
Deploying agoric-sdk with Cloudflare Pages
|
@@ -50,44 +51,89 @@ export const OrchestratorI = M.interface('Orchestrator', { | |||
* orchestrationService: Remote<OrchestrationService>; | |||
* storageNode: Remote<StorageNode>; | |||
* timerService: Remote<TimerService>; | |||
* vowTools: VowTools; | |||
* zcf: ZCF; | |||
* }} powers | |||
*/ | |||
export const prepareOrchestrator = ( |
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.
this is now a kit. prepareOrchestratorKit
.
but no consumers need these other facets so the module should still export a prepareOrchestrator
that returns a makeOrchestrator
that returns just its orchestrator
facet
packages/orchestration/src/facade.js
Outdated
const orc = makeOrchestrator().orchestrator; | ||
|
||
return async (...args) => fn(orc, ctx, ...args); |
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.
this shouldn't have access to the whole kit (per above).
b3802fc
to
5b3d56e
Compare
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.
Requested changes made, sending to merge queue
- returning unwrapped vows to ensure tests are passing and functionality is preserved
5b3d56e
to
94d0c3c
Compare
refs: #9449
Description
Returns unwrapped vows in
Orchestrator
(makeLocalAccount
,getChain
).