Skip to content
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

WIP: Charge fees and meter #3674

Closed
wants to merge 17 commits into from
Closed

WIP: Charge fees and meter #3674

wants to merge 17 commits into from

Conversation

katelynsills
Copy link
Contributor

Work in progress.

Stacked on #3663

* BREAKING CHANGE: make the run mint within Zoe, and give only the treasury the ability to create a ZCFMint with it

* chore: change 'makeZoe' to 'makeZoeKit'

* chore: add "shutdownZoeVat" argument to Zoe, and pass it to `makeIssuerKit` for invitation issuerKit and fee issuerKit

* chore: manually lint-fix install-on-chain.js

See #3672 for the issue to fix the root problem
@katelynsills katelynsills added the Zoe package: Zoe label Aug 13, 2021
@katelynsills katelynsills added this to the Testnet: Metering Phase milestone Aug 13, 2021
@katelynsills katelynsills self-assigned this Aug 13, 2021
@katelynsills katelynsills force-pushed the charge-fees-and-meter branch from 8f38ef8 to 3e1bbad Compare August 13, 2021 16:29
@katelynsills katelynsills mentioned this pull request Aug 13, 2021
@katelynsills katelynsills force-pushed the charge-fees-and-meter branch from 5e23acd to 55689e1 Compare August 13, 2021 20:44
@katelynsills katelynsills force-pushed the charge-fees-and-meter branch from 55689e1 to 225e417 Compare August 14, 2021 00:38
@katelynsills katelynsills force-pushed the charge-fees-and-meter branch from 225e417 to 175e764 Compare August 14, 2021 01:12
@katelynsills katelynsills force-pushed the charge-fees-and-meter branch from 175e764 to ac81513 Compare August 14, 2021 01:20
@katelynsills
Copy link
Contributor Author

Closed in favor of #3319

@katelynsills katelynsills deleted the charge-fees-and-meter branch August 17, 2021 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Zoe package: Zoe
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants