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

Update staking-governance-supply.md #1748

Merged
merged 2 commits into from
Oct 14, 2024
Merged

Update staking-governance-supply.md #1748

merged 2 commits into from
Oct 14, 2024

Conversation

nwhite1
Copy link
Member

@nwhite1 nwhite1 commented Oct 2, 2024

Clarifying unlock dates

Clarifying unlock dates
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Outside diff range and nitpick comments (1)
learn/staking-governance-supply.md (1)

104-105: Excellent addition for clarity on unlock dates

This note provides valuable clarification on the exact dates for yearly unlocks, taking into account the leap year in 2024. It helps prevent any potential confusion about the precise timing of unlock events.

A minor suggestion to enhance readability:

-_Note: Due to 2024 being a leap year, the yearly unlock intervals will occur on October 30th of each year. For example, unlocks at year 1 will occur on October 30, 2024._
+_Note: Due to 2024 being a leap year, the yearly unlock intervals will occur on October 30th of each year. For example, the first yearly unlock (year 1) will occur on October 30, 2024._

This small change makes it clearer that October 30, 2024, is specifically the date for the first yearly unlock.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between c1b440c and de6142c.

📒 Files selected for processing (1)
  • learn/staking-governance-supply.md (1 hunks)
🔇 Additional comments (1)
learn/staking-governance-supply.md (1)

Line range hint 1-105: Overall document review: Approved with suggestions

The document provides a comprehensive and well-structured overview of staking, governance, and supply for the Celestia blockchain. The added note about unlock dates significantly improves clarity. If the suggested updates regarding specific dates in the unlock schedules are implemented, the document will offer even more precise and consistent information to readers.

Great job on maintaining and improving this crucial documentation!

Comment on lines +104 to +105
_Note: Due to 2024 being a leap year, the yearly unlock intervals will occur on October 30th of each year. For example, unlocks at year 1 will occur on October 30, 2024._

Copy link
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Consider updating unlock schedules with specific dates

The added note provides excellent clarity on the yearly unlock dates. To maintain consistency throughout the document, consider updating the unlock schedule table and the "Unlocks" section with these specific dates. This would provide readers with a clear and consistent understanding of when unlocks occur across all sections of the document.

For example, in the unlock schedule table, you could update entries like this:

- 33.33% unlocked at year 1.<br/>Remaining 66.67% unlocks continuously from year 1 to year 2.
+ 33.33% unlocked on October 30, 2024.<br/>Remaining 66.67% unlocks continuously from October 30, 2024 to October 30, 2025.

This level of detail would align with the newly added note and provide readers with precise information throughout the document.

Committable suggestion was skipped due to low confidence.

Copy link
Contributor

@rootulp rootulp left a comment

Choose a reason for hiding this comment

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

LGTM

@celestiaorg celestiaorg deleted a comment from coderabbitai bot Oct 14, 2024
@rootulp rootulp merged commit 9e2304a into celestiaorg:main Oct 14, 2024
3 of 4 checks passed
Copy link

gitpoap-bot bot commented Oct 14, 2024

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2024 Celestia Contributor:

GitPOAP: 2024 Celestia Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

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.

4 participants