Skip to content

fix: simplify runtime notifications - #337

Merged
Pigbibi merged 1 commit into
mainfrom
fix/concise-runtime-notifications-20260926
Sep 26, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
fix/concise-runtime-notifications-20260926

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep user notifications focused on total equity, nonzero holdings, supplemental content, and trade or cycle result
  • add a shared compact-section adapter with explicit localized supplemental lines
  • update paper preview output and prevent Chinese and English unit mixing

Verification

  • 47 tests passed
  • Ruff passed
  • git diff --check passed

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 1f8df52 into main Sep 26, 2026
1 check passed
@Pigbibi
Pigbibi deleted the fix/concise-runtime-notifications-20260926 branch September 26, 2026 14:56
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