-
Notifications
You must be signed in to change notification settings - Fork 3.6k
additionaldata-message-exception #12819
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
Enhanced exception handling in messaging import service by including the original exception object in additionalData for better debugging capabilities.
- Added original exception details to
handleUnknownException
inpackages/twenty-server/src/modules/messaging/message-import-manager/services/messaging-import-exception-handler.service.ts
for improved error tracing - Modification aligns with error handling best practices by preserving complete error context for debugging
1 file reviewed, no comments
Edit PR Review Bot Settings | Greptile
📊 API Changes ReportREST API ChangesSummary🔄 Changed Operations (63)
|
🚀 Preview Environment Ready! Your preview environment is available at: http://bore.pub:57957 This environment will automatically shut down when the PR is closed or after 5 hours. |
better loging exceptions by keeping the orginal error (otherwise it's swallowed and hard to debug precisely)
better loging exceptions by keeping the orginal error (otherwise it's swallowed and hard to debug precisely)