Skip to content

Refine and extend Node monitoring section of README#476

Merged
hannahhoward merged 5 commits intofilecoin-saturn:mainfrom
31z4:monitoring-docs
Dec 1, 2023
Merged

Refine and extend Node monitoring section of README#476
hannahhoward merged 5 commits intofilecoin-saturn:mainfrom
31z4:monitoring-docs

Conversation

@31z4
Copy link
Copy Markdown
Contributor

@31z4 31z4 commented Aug 27, 2023

I removed https://orchestrator.strn.pl/stats because it redirects to https://dashboard.saturn.tech/stats, which is already mentioned. I also added links to the Saturn Explorer and Saturn Moonlet. The latter is the project that I built and maintain ☺️

@vorburger
Copy link
Copy Markdown
Contributor

LGTM

@31z4
Copy link
Copy Markdown
Contributor Author

31z4 commented Aug 29, 2023

@DiegoRBaquero please have a look when you have time. Thanks!

@vorburger
Copy link
Copy Markdown
Contributor

@31z4 FYI #481 (comment) "Given Moonlet is external, it should not be mentioned in the official L1 FAQ" ... 😿

@31z4
Copy link
Copy Markdown
Contributor Author

31z4 commented Aug 31, 2023

@vorburger I think Diego's comment is relevant specifically to the FAQ section of the docs. In general, I think node operators should know about non-official monitoring tools built by the community.

Comment thread README.md
31z4 and others added 3 commits October 20, 2023 10:17
Co-authored-by: Hannah Howard <hannah@hannahhoward.net>
Signed-off-by: Elisey Zanko <elisey.zanko@gmail.com>
@31z4
Copy link
Copy Markdown
Contributor Author

31z4 commented Oct 23, 2023

@hannahhoward I accepted your suggestion and updated Table of Contents. Also, @DiegoRBaquero approved already. Can you please merge now?

@31z4 31z4 requested a review from hannahhoward October 25, 2023 05:24
@hannahhoward hannahhoward enabled auto-merge (squash) December 1, 2023 01:00
@hannahhoward hannahhoward merged commit a374804 into filecoin-saturn:main Dec 1, 2023
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