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

1426 Cleanup old Highway and rewards references #1434

Merged
merged 24 commits into from
May 3, 2024

Conversation

@ipopescu ipopescu changed the title 1426 cleanup 1426 cleanup old Highway and rewards references Apr 16, 2024
@ipopescu ipopescu changed the title 1426 cleanup old Highway and rewards references 1426 Cleanup old Highway and rewards references Apr 16, 2024
Copy link
Contributor

@AlexanderLimonov AlexanderLimonov left a comment

Choose a reason for hiding this comment

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

Some relatively small changes are needed I think, but there's also further work on other aspects of 2.0 economics that might need to go in a separate PR

source/docs/casper/concepts/economics/consensus.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/consensus.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/consensus.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/index.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/runtime.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/staking/concepts.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/staking/concepts.md Outdated Show resolved Hide resolved
source/docs/casper/concepts/economics/staking/concepts.md Outdated Show resolved Hide resolved
@ipopescu
Copy link
Collaborator Author

ipopescu commented May 1, 2024

@AlexanderLimonov I have addressed all the comments, with the exception of these 3, where I am still following up:

New content explaining incentives, gas, and fees should be added or linked using a new PR -- unless it can be done quickly at a summary level.

@ipopescu
Copy link
Collaborator Author

ipopescu commented May 2, 2024

Resolved 2/3 of the open comments:

Copy link
Contributor

@ACStone-MTS ACStone-MTS left a comment

Choose a reason for hiding this comment

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

LGTM!

@ipopescu ipopescu merged commit 65cae43 into casper-network:feat-2.0_docs May 3, 2024
8 checks passed
@ipopescu ipopescu deleted the 1426_cleanup branch May 3, 2024 13:27
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.

3 participants