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

feat: add contacts liveness service to base layer wallet #3857

Merged

Conversation

hansieodendaal
Copy link
Contributor

@hansieodendaal hansieodendaal commented Feb 19, 2022

Description

Implemented a liveness service for the base layer wallet's contacts that will send ping messages and respond with pong messages to indicate liveness information.

The next aspect of contacts liveness will be to integrate the events into the various wallet front ends.

Motivation and Context

There is a need to know which managed contacts in a wallet are online and when they have last been seen.

How Has This Been Tested?

  • [WIP] Additional unit tests
  • System level testing between various console wallets

Copy link
Contributor

@philipr-za philipr-za left a comment

Choose a reason for hiding this comment

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

Looking good so far

base_layer/wallet/src/config.rs Outdated Show resolved Hide resolved
base_layer/wallet/src/config.rs Outdated Show resolved Hide resolved
@hansieodendaal hansieodendaal changed the title [WIP] feat: add contacts liveness service to base layer wallet feat: add contacts liveness service to base layer wallet Feb 21, 2022
Implemented a liveness service for the base layer wallet's contacts
that wil send ping messages and respond with pong messages to indicate
liveness information.
@aviator-app aviator-app bot added mq-failed and removed mq-failed labels Feb 22, 2022
@aviator-app aviator-app bot merged commit 0d96ea3 into tari-project:development Feb 22, 2022
@hansieodendaal hansieodendaal deleted the ho_wallet_contacts_status branch February 23, 2022 00:24
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