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

Remove presence notifications #370

Closed
futurechimp opened this issue Oct 12, 2020 · 2 comments · Fixed by #401
Closed

Remove presence notifications #370

futurechimp opened this issue Oct 12, 2020 · 2 comments · Fixed by #401
Assignees
Milestone

Comments

@futurechimp
Copy link
Contributor

We should remove presence notifications from all Rust nodes, we can now track things through node announcements in the blockchain.

@futurechimp futurechimp added this to the 0.9.0 milestone Oct 12, 2020
@futurechimp futurechimp added this to Backlog in Core systems via automation Oct 12, 2020
@futurechimp futurechimp self-assigned this Oct 13, 2020
@futurechimp
Copy link
Contributor Author

I'll also include removing presence from directory stuff in the validators here. When this is done, all presence notifications will be gone from the full system.

@futurechimp
Copy link
Contributor Author

This is now gone from validators and the metrics server. Still need to remove all presence-related stuff from the mixnet nodes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Core systems
  
Done
Development

Successfully merging a pull request may close this issue.

2 participants