Skip to content

refactor(notifications): migrate to shared renderer_base#208

Merged
Pigbibi merged 3 commits into
mainfrom
refactor/notification-unification
Jul 8, 2026
Merged

refactor(notifications): migrate to shared renderer_base#208
Pigbibi merged 3 commits into
mainfrom
refactor/notification-unification

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Replace duplicated formatting/volatility helpers with imports from quant_platform_kit.notifications.renderer_base.

Depends on: QuantPlatformKit#200

Changes

  • notifications/telegram.py: 1270 → 1097 lines (-173)
  • Deleted local copies of shared helpers
  • Kept Firstrade-specific _format_tqqq_risk_control_lines thin wrapper
  • All platform-specific functions preserved

🤖 Generated with Claude Code

Replace duplicated formatting/volatility helpers with imports from
quant_platform_kit.notifications.renderer_base.

Co-Authored-By: Claude <noreply@anthropic.com>
@chatgpt-codex-connector

This comment has been minimized.

@cursor

cursor Bot commented Jul 8, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Co-Authored-By: Claude <noreply@anthropic.com>
@cursor

cursor Bot commented Jul 8, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Pigbibi Pigbibi enabled auto-merge (squash) July 8, 2026 21:18
Co-Authored-By: Claude <noreply@anthropic.com>
@cursor

cursor Bot commented Jul 8, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Pigbibi Pigbibi merged commit 2d50452 into main Jul 8, 2026
2 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.

1 participant