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 nodes.go with SafetyNet node #1437

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

carry2web
Copy link

We are online for several months with three validators. Adding to deso nodes list

@carry2web carry2web requested a review from a team as a code owner December 11, 2024 21:40
@StarGeezerPhil
Copy link

StarGeezerPhil commented Dec 21, 2024

Just to support @carry2web's PR - it'd be great to get the first of the SafetyNet nodes included in the official list.

EDIT: I'll post the suggested changes as a separate PR.

@carry2web
Copy link
Author

@lazynina @diamondhands0 if you honor #1438 this one can be skipped as @StarGeezerPhil included SafetyNet in his improved proposal. Thank you

@StarGeezerPhil
Copy link

@lazynina @diamondhands0 if you honor #1438 this one can be skipped as @StarGeezerPhil included SafetyNet in his improved proposal. Thank you

Thanks @carry2web, I've also made comment on-chain about the prior proposals that were skipped:

If the team are happy to accept #1438 I'm happy to update it in chronological order so that Nathan's much earlier request can be honoured for node ID 30, which he has been using for some considerable time already.

This would push Focus to 31, SafetyNet to 32 and so on.

On-chain post requesting this is considered:
https://diamondapp.com/posts/5f8d804ca58a6263d93d58fb4cf5ab3cc58d84bd9f2e091f04638832af1074cd

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.

2 participants