Skip to content

fix(charts): fall back to document delivery for chart photos - #34

Merged
scottishwidow merged 1 commit into
mainfrom
chart-document-fallback
May 1, 2026
Merged

fix(charts): fall back to document delivery for chart photos#34
scottishwidow merged 1 commit into
mainfrom
chart-document-fallback

Conversation

@scottishwidow

Copy link
Copy Markdown
Owner

Summary

  • Fall back to Telegram document delivery when chart photo delivery raises a Telegram error.
  • Restrict photo attempts to Telegram-friendly raster chart MIME types and send other chart files as documents.
  • Mark the local chart-preview feedback issue complete and add handler regression coverage.

Test Plan

  • uv run pytest
  • uv run ruff check .
  • uv run ruff format --check .
  • uv run mypy
  • uv run pre-commit run --all-files

@scottishwidow
scottishwidow merged commit e2e98d4 into main May 1, 2026
4 checks passed
@scottishwidow
scottishwidow deleted the chart-document-fallback branch May 1, 2026 18:39
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