Skip to content

[WIP] [WIP] Address feedback on per-message export feature#796

Merged
eldong merged 2 commits intoexport-messagefrom
copilot/sub-pr-783-another-one
Mar 12, 2026
Merged

[WIP] [WIP] Address feedback on per-message export feature#796
eldong merged 2 commits intoexport-messagefrom
copilot/sub-pr-783-another-one

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

  • Identify the error handling issue in route_backend_conversation_export.py
  • Fix api_export_conversations: return generic error, keep str(e) for debug only
  • Fix api_export_message_word: return generic error, keep str(e) for debug only
  • Run code review and security checks (0 alerts)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@eldong
Copy link
Collaborator

eldong commented Mar 12, 2026

Set the billing

@eldong eldong marked this pull request as ready for review March 12, 2026 20:11
@eldong
Copy link
Collaborator

eldong commented Mar 12, 2026

I set the billing

@eldong
Copy link
Collaborator

eldong commented Mar 12, 2026

@copilot apply the requested changes:

  • return a general error message to the client
  • keep str(e) only for debug/logging

…r(e) for debug only

Co-authored-by: eldong <11573590+eldong@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 12, 2026

@copilot apply the requested changes:

  • return a general error message to the client
  • keep str(e) only for debug/logging

Done in commit 93eac3e. Both export endpoints (api_export_conversations and api_export_message_word) now return the generic message "Export failed due to a server error. Please try again later." to the client, while str(e) is kept exclusively in debug_print() for server-side logging.

@eldong eldong merged commit 94fffb1 into export-message Mar 12, 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.

2 participants