Skip to content

Comments

feat: Allow adding and removing controllers#1950

Merged
adamspofford-dfinity merged 8 commits intomasterfrom
adamspofford/controller-mgmt
Dec 9, 2021
Merged

feat: Allow adding and removing controllers#1950
adamspofford-dfinity merged 8 commits intomasterfrom
adamspofford/controller-mgmt

Conversation

@adamspofford-dfinity
Copy link
Contributor

Fixes SDK-207

@adamspofford-dfinity adamspofford-dfinity requested a review from a user December 6, 2021 23:34
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Please add an entry to the changelog regarding the wallet.wasm version change (see changelog for other examples)

Also, we should probably also increment the wallet patch version in the cycles-wallet repo.

@adamspofford-dfinity adamspofford-dfinity enabled auto-merge (squash) December 8, 2021 23:10
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Please add a note to the changelog about this. Approving under the assumption that this is done.

@adamspofford-dfinity adamspofford-dfinity merged commit 3eca212 into master Dec 9, 2021
@adamspofford-dfinity adamspofford-dfinity deleted the adamspofford/controller-mgmt branch December 9, 2021 18:31
adamspofford-dfinity added a commit that referenced this pull request Dec 9, 2021
adamspofford-dfinity added a commit that referenced this pull request Dec 10, 2021
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.

1 participant