Skip to content

Comments

Feature: Indefinite duration Boombox#10

Open
friedger wants to merge 12 commits intomainfrom
feat/indefinite-duration
Open

Feature: Indefinite duration Boombox#10
friedger wants to merge 12 commits intomainfrom
feat/indefinite-duration

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