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.

Sign up for GitHub

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

Jump to bottom

Updates for executed proposals #450

Merged
merged 3 commits into from
Jun 25, 2024
Merged

Updates for executed proposals #450

merged 3 commits into from
Jun 25, 2024

Conversation

jmrossy
Copy link
Collaborator

@jmrossy jmrossy commented Jun 24, 2024 •
edited
Loading

  • Add executed dates for some new proposals
  • Add test to catch duplicated proposal IDs
  • Fix some incorrect proposal IDs

See #394 for how I got these values

@jmrossy jmrossy requested a review from martinvol June 24, 2024 21:23
@0xzoz 0xzoz merged commit 113a533 into main Jun 25, 2024
5 checks passed
@0xzoz 0xzoz deleted the rossy/cgp-fixes-24062024 branch June 25, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xzoz 0xzoz 0xzoz approved these changes

@0xGoldo 0xGoldo 0xGoldo approved these changes

@work-0xj4an work-0xj4an Awaiting requested review from work-0xj4an

@ericnakagawa ericnakagawa Awaiting requested review from ericnakagawa

@martinvol martinvol Awaiting requested review from martinvol

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jmrossy @0xzoz @0xGoldo