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

Refactor events listening logic and add key registry listener #5

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

haardikk21
Copy link
Contributor

  • handle ChangeRecoveryAddress event from IdRegistry
  • add key registry listener, refactor events listening code

@@ -2,6 +2,8 @@ extern crate prost_build;
use std::path::PathBuf;

fn main() {
println!("cargo:rerun-if-changed=../../protobufs");
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we need the print statement here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, weird rust thing - it was a TIL

https://doc.rust-lang.org/cargo/reference/build-scripts.html

Effectively caches the build unless something in that dir changes

@avichalp
Copy link
Collaborator

@haardikk21: Looks good to me! I think, we can merge this. I will work on refactoring the event listeners to abstract out the repeated code in there in a different PR.

@haardikk21
Copy link
Contributor Author

@haardikk21: Looks good to me! I think, we can merge this. I will work on refactoring the event listeners to abstract out the repeated code in there in a different PR.

Sounds good. FWIW, there's some TODO comments to add logic beyond just persisting the event in the key registry listener, and then one more listener left to do iirc.

I can tackle that later or if you want to, feel free to do that

@haardikk21 haardikk21 merged commit 033ecb0 into main Dec 12, 2023
2 of 4 checks passed
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