-
Notifications
You must be signed in to change notification settings - Fork 1.7k
UBERF-12966: Send messages from Gmail threads #9657
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
Signed-off-by: Artem Savchenko <armisav@gmail.com>
|
Connected to Huly®: UBERF-13042 |
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.
Pull Request Overview
This PR implements the ability to send messages from Gmail threads as part of UBERF-12966. The feature allows outgoing Gmail messages to be sent when messages are created in Huly threads that are connected to Gmail channels.
Key changes include:
- Added message sending capabilities from Gmail threads with proper header identification
- Implemented markdown to HTML conversion for email formatting
- Added queue consumer to listen for new message events and trigger Gmail sending
Reviewed Changes
Copilot reviewed 19 out of 21 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| services/mail/mail-common/src/utils.ts | Added utility functions for message metadata, sync detection, and reply subject generation |
| services/mail/mail-common/src/types.ts | Extended types to support message metadata and communication domain |
| services/mail/mail-common/src/txHandler.ts | Added transaction handler to convert platform transactions to message events |
| services/mail/mail-common/src/send.ts | Added placeholder functions for message sending logic |
| services/mail/mail-common/src/person.ts | Enhanced person cache with email lookup by social ID |
| services/mail/mail-common/src/message.ts | Updated message creation to include extra metadata |
| services/mail/mail-common/src/md.ts | Added markdown to HTML conversion utilities |
| services/mail/mail-common/src/index.ts | Exported new modules |
| services/mail/mail-common/src/tests/md.test.ts | Added comprehensive tests for markdown conversion |
| services/mail/mail-common/package.json | Added markdown-it dependency |
| services/gmail/pod-gmail/src/workspaceClient.ts | Added message handling delegation to Gmail clients |
| services/gmail/pod-gmail/src/utils.ts | Added Gmail search query builder for message deduplication |
| services/gmail/pod-gmail/src/types.ts | Added Gmail-specific constants and headers |
| services/gmail/pod-gmail/src/message/v2/send.ts | Implemented Gmail message sending with proper headers and formatting |
| services/gmail/pod-gmail/src/message/v2/message.ts | Enhanced message processing to skip Huly-sent messages |
| services/gmail/pod-gmail/src/gmailController.ts | Added queue consumer for handling new message events |
| services/gmail/pod-gmail/src/gmail.ts | Implemented core Gmail message sending logic with deduplication |
| services/gmail/pod-gmail/src/client.ts | Added communication client creation utilities |
| services/gmail/pod-gmail/package.json | Added required dependencies |
Files not reviewed (1)
- common/config/rush/pnpm-lock.yaml: Language not supported
Signed-off-by: Artem Savchenko <armisav@gmail.com>
…cter sanitization Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> Signed-off-by: Artyom Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
Signed-off-by: Artem Savchenko <armisav@gmail.com>
No description provided.