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

180 Update documentation content #181

Merged

Conversation

nevendyulgerov
Copy link
Contributor

@nevendyulgerov nevendyulgerov commented Apr 23, 2024

Closes #180.

@brunomenezes
Copy link

@nevendyulgerov, I think you need to include a reviewer here. Maybe @masiedu4 could help with this PR?

@masiedu4 masiedu4 self-requested a review May 6, 2024 11:18
@masiedu4
Copy link
Contributor

masiedu4 commented May 6, 2024

@nevendyulgerov, I think you need to include a reviewer here. Maybe @masiedu4 could help with this PR?

Approved. You can merge now!

@nevendyulgerov
Copy link
Contributor Author

Hey @masiedu4 , @brunomenezes , I don't have write access to this repo so I cannot add reviewers or merge PRs. If possible, please merge this one on my behalf.

@brunomenezes
Copy link

@nevendyulgerov, I think you need to include a reviewer here. Maybe @masiedu4 could help with this PR?

Approved. You can merge now!

Nice! Hey @masiedu4, we don't have access to merge the PR. Also, @nevendyulgerov did not even have access rights to add a reviewer.

Also, one question: how is the release process once it is merged? I am asking because once it goes live, I will ping Karmen Truong to have a look.

@masiedu4 masiedu4 merged commit d0985b0 into cartesi:main May 7, 2024
@masiedu4
Copy link
Contributor

masiedu4 commented May 7, 2024

@nevendyulgerov, I think you need to include a reviewer here. Maybe @masiedu4 could help with this PR?

Approved. You can merge now!

Nice! Hey @masiedu4, we don't have access to merge the PR. Also, @nevendyulgerov did not even have access rights to add a reviewer.

Also, one question: how is the release process once it is merged? I am asking because once it goes live, I will ping Karmen Truong to have a look.

Hi @brunomenezes all the changes are live. Thanks! https://docs.cartesi.io/earn-ctsi/staking/

@brunomenezes
Copy link

brunomenezes commented May 7, 2024

Hi @brunomenezes all the changes are live. Thanks! https://docs.cartesi.io/earn-ctsi/staking/

Thanks @masiedu4! you're a rockstar 🚀

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.

Update documentation content
3 participants