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

CCIP-019: PoX 4 update #35

Merged
merged 23 commits into from
Aug 9, 2024
Merged

CCIP-019: PoX 4 update #35

merged 23 commits into from
Aug 9, 2024

Conversation

friedger
Copy link
Contributor

@friedger friedger commented May 4, 2024

This CCIP ensure to continue the rules defined by the DAO by updating from pox-3 to pox-4.

Preview: https://github.com/citycoins/governance/blob/feat/add-ccip-019/ccips/ccip-019/ccip-019-mia-pox-4-stacking.md

@friedger friedger changed the title PoX 4 update CCIP-019: PoX 4 update May 5, 2024
Copy link
Contributor

@whoabuddy whoabuddy left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Added my comments following the review of citycoins/protocol#67 in tandem.

ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
ccips/ccip-019/ccip-019-pox-4-stacking.md Outdated Show resolved Hide resolved
@friedger friedger requested a review from whoabuddy May 13, 2024 13:09
@whoabuddy whoabuddy assigned whoabuddy and unassigned whoabuddy Jul 30, 2024
@whoabuddy
Copy link
Contributor

@friedger any blockers to the changes I made in #42?

@whoabuddy
Copy link
Contributor

I think we're good to go on the proposal side, next up will be fixing a few things in the implementation per this comment. Going to leave this open and give it a final once-over once contract is ready for deployment.

@whoabuddy whoabuddy merged commit 4643440 into main Aug 9, 2024
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