Skip to content

feat: implement investor notification system for score changes#154

Open
NnamdiCyber wants to merge 2 commits into
Heliobond:mainfrom
NnamdiCyber:feat/investor-notification-system
Open

feat: implement investor notification system for score changes#154
NnamdiCyber wants to merge 2 commits into
Heliobond:mainfrom
NnamdiCyber:feat/investor-notification-system

Conversation

@NnamdiCyber

Copy link
Copy Markdown

Closes #131

Changes

On-Chain: Enhanced ScoreChanged Event

  • Added ScoreChanged event to project_registry/src/events.rs with old/new score values and interest rates
  • Emitted from both update_impact_score and update_credit_quality_score
  • No-op identical updates skip event emission

Off-Chain: Notification Service (notification-service/)

  • Event listener polling Soroban RPC for ScoreChanged events
  • Email notifications via nodemailer (SMTP)
  • Webhook notifications via HTTP POST
  • SQLite-backed investor preference store
  • REST API at /preferences/:address for CRUD operations
  • Docker support

Documentation

  • docs/NOTIFICATIONS.md — full system documentation
  • Updated README.md, CONTRACTS.md, docs/INTEGRATION.md

Emit ScoreChanged event with old/new scores and rates from
update_impact_score and update_credit_quality_score. Add off-chain
notification service with email+webhook dispatch, REST API for
preferences, and full documentation.

Closes Heliobond#131
@drips-wave

drips-wave Bot commented Jun 27, 2026

Copy link
Copy Markdown

@NnamdiCyber Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@AbelOsaretin

Copy link
Copy Markdown
Contributor

Resolve merge conflict.

@NnamdiCyber

Copy link
Copy Markdown
Author

Resolve merge conflict.

The merge conflict is resolved

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.

[Feature] Implement investor notification system for score changes

2 participants