Add handlers for unsupported content types in private chats

[karilaa-dev]

PR Type

Enhancement

---

Description

• Add handlers for unsupported content types in private chats

• Improve error messaging for invalid links and uploads

• Update locale strings with more user-friendly messages

• Fix reaction emoji on errors in group chats

---

Diagram Walkthrough

flowchart LR
  A[User sends message] --> B{Content type?}
  B -->|Video| C[Reply: video_upload_hint]
  B -->|Photo| D[Reply: tiktok_links_only]
  B -->|Voice/Audio| D
  B -->|Text| E{Valid TikTok link?}
  E -->|No URL| F[Reply: send_link_prompt]
  E -->|Non-TikTok URL| G[Reply: non_tiktok_link]
  E -->|Valid| H[Process video]
  B -->|Other| D

Loading

File Walkthrough

	Relevant files
Enhancement	1 files get_video.py Add handlers for unsupported content types                              +72/-5
Documentation	8 files ar.json Update Arabic error messages                                                          +9/-5      en.json Update English error messages                                                        +9/-5      hi.json Update Hindi error messages                                                            +9/-5      id.json Update Indonesian error messages                                                  +9/-5      ru.json Update Russian error messages                                                        +9/-5      so.json Update Somali error messages                                                          +9/-5      uk.json Update Ukrainian error messages                                                    +9/-5      vi.json Update Vietnamese error messages                                                  +9/-5

---

[zam-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Code Duplication The handlers handle_video_upload, handle_image_upload, and handle_voice_upload contain nearly identical code for checking chat type, retrieving user settings, and getting the language. This logic should be extracted into a helper function to reduce duplication and improve maintainability. @video_router.message(F.video | F.video_note) async def handle_video_upload(message: Message): """Inform users that we need links, not video uploads (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["video_upload_hint"]) @video_router.message(F.photo) async def handle_image_upload(message: Message): """Inform users that we only support TikTok links, not images (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["tiktok_links_only"]) @video_router.message(F.voice | F.audio) async def handle_voice_upload(message: Message): """Inform users that we only support TikTok links, not voice messages (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["tiktok_links_only"]) Catch-All Handler Risk The handle_unsupported_content handler is registered with @video_router.message() without any filters. This catch-all handler may interfere with other message handlers or commands in the router. Consider adding more specific filters or ensuring this handler is registered at the appropriate priority level. @video_router.message() async def handle_unsupported_content(message: Message): """Catch-all for any unsupported content types (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["tiktok_links_only"]) Missing Error Handling The new handlers (handle_video_upload, handle_image_upload, handle_voice_upload, and handle_unsupported_content) lack try-except blocks around database calls (get_user_settings) and message replies. If these operations fail, users won't receive any feedback. Consider adding error handling to provide a better user experience. @video_router.message(F.video | F.video_note) async def handle_video_upload(message: Message): """Inform users that we need links, not video uploads (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["video_upload_hint"]) @video_router.message(F.photo) async def handle_image_upload(message: Message): """Inform users that we only support TikTok links, not images (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["tiktok_links_only"]) @video_router.message(F.voice | F.audio) async def handle_voice_upload(message: Message): """Inform users that we only support TikTok links, not voice messages (private chats only).""" if message.chat.type != "private": return settings = await get_user_settings(message.chat.id) if settings: lang = settings[0] else: lang = await lang_func(message.chat.id, message.from_user.language_code, True) await message.reply(locale[lang]["tiktok_links_only"])

[zam-review]

Failed to generate code suggestions for PR