Skip to content

feat: 🔈 Improve logs for Blockchain Service processing of notifications #631

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.

Sign up for GitHub

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

Jump to bottom
Draft
ffarall wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

feat: 🔈 Improve logs for Blockchain Service processing of notifications #631

ffarall wants to merge 6 commits into from
+199 −184

Conversation

@ffarall
Copy link
Collaborator

@ffarall ffarall commented Dec 24, 2025 •
edited
Loading

  1. Add logs at the end of import block processing, finality block processing, and initial sync block processing.
  2. Unify spelling of finalise vs finalize.

@ffarall ffarall added B0-silent Changes should not be mentioned in any release notes D2-noauditneeded🙈 PR doesn't need to be audited not-breaking Does not need to be mentioned in breaking changes labels Dec 24, 2025
@ffarall ffarall changed the title feat: 🔈 Improve logs for Blockchain Service processing of events feat: 🔈 Improve logs for Blockchain Service processing of notifications Dec 24, 2025
@santikaplan santikaplan self-assigned this Dec 26, 2025
@ffarall ffarall requested a review from santikaplan December 26, 2025 13:18
santikaplan
santikaplan approved these changes Dec 26, 2025
View reviewed changes
Copy link
Collaborator

@santikaplan santikaplan left a comment

Choose a reason for hiding this comment

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

Sweet!

@ffarall ffarall marked this pull request as draft December 26, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@santikaplan santikaplan santikaplan approved these changes

Assignees

@santikaplan santikaplan

Labels

B0-silent Changes should not be mentioned in any release notes D2-noauditneeded🙈 PR doesn't need to be audited not-breaking Does not need to be mentioned in breaking changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ffarall @santikaplan