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

Feature: Indefinite duration Boombox #10

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

friedger
Copy link
Member

@friedger friedger commented Jan 14, 2022

This PR changes the boombox-admin protocol to support indefinite duration and any cycle to join

  • add extend-boomboxing
  • add get-total-stacked-by-cycle
  • add tests

Suggested naming:

classic Boombox: members can only join at a given cycle and for 1 locking period
indefinite Boombox: members can join at any cycle and continue after the first locking period
one-shot Boombox: members can join at any cycle, but only for 1 locking period
long Boombox: members can only join at a given cycle and continue after the first locking period

@friedger friedger marked this pull request as ready for review January 25, 2022 10:18
@friedger friedger changed the title [WIP] Feature: Indefinite duration Feature: Indefinite duration Boombox Jan 25, 2022
@friedger friedger changed the base branch from main to chore/friedger-monorepo February 3, 2022 22:24
@dantrevino dantrevino requested a review from amrro February 13, 2022 17:23
}
import {
stackAggregationCommit,
poxAllowContractCaller,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to run clarinet test insdide boombox-admin package and I got this error.

TS2305 [ERROR]: Module '"file:///Users/amrro/Developer/boom/boom-contracts/packages/boombox-admin/tests/client/boombox-admin.ts"' has no exported member 'poxAllowContractCaller'.
  poxAllowContractCaller,
  ~~~~~~~~~~~~~~~~~~~~~~

@friedger friedger changed the base branch from chore/friedger-monorepo to main February 19, 2023 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants