-
Notifications
You must be signed in to change notification settings - Fork 35
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
Claim hermes tokens #1090
Claim hermes tokens #1090
Conversation
46f76f4
to
a736dc2
Compare
a736dc2
to
805e603
Compare
Is this good to merge? It's still in draft. |
Left in draft because we're on an rc for Fedimint, not sure what our strategy is here Also haven't tested the zap decoding stuff |
needs rebase |
805e603
to
df77624
Compare
mutiny-core/src/hermes.rs
Outdated
"Received DM for unknown federation {}", | ||
notification.federation_id | ||
); | ||
// todo should we save this to storage? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Save what in storage? The failed DM?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit on a random comment, otherwise gtg
df77624
to
fe92320
Compare
Part of #1079