Skip to content

Record Firstrade notification suppression in reports#91

Merged
Pigbibi merged 1 commit into
mainfrom
codex/firstrade-notification-report-fields
Jun 18, 2026
Merged

Record Firstrade notification suppression in reports#91
Pigbibi merged 1 commit into
mainfrom
codex/firstrade-notification-report-fields

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • include notification_sent and notification_suppressed in Firstrade runtime report summaries
  • carry notification_error into the report summary when notification delivery fails

Tests

  • git diff --check
  • python -m py_compile main.py

@Pigbibi Pigbibi merged commit e412631 into main Jun 18, 2026
1 check passed
@Pigbibi Pigbibi deleted the codex/firstrade-notification-report-fields branch June 18, 2026 07:25
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.

1 participant