Skip to content
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

Integrate ActionMailbox #8049

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Integrate ActionMailbox #8049

wants to merge 2 commits into from

Conversation

gbp
Copy link
Member

@gbp gbp commented Dec 15, 2023

Relevant issue(s)

What does this do?

Integrate ActionMailbox

Why was this needed?

Integrate ActionMailbox into RequestMailer.receive for inbound email handling. This maintains existing functionality but lays the groundwork for future enhancements.

Implementation notes

Added an origin column to the ActionMailbox::InboundEmail table. This change was necessary to preserve the functionality of checking the mail source, avoiding a naming clash with the source method.

Benefits of using ActionMailbox include:

  1. Enhanced email routing capabilities, enabling specialized processing like the Excel hidden data spreadsheet analyzer.
  2. Refactoring opportunities for RequestMailer#receive, particularly for spam detection, duplicate email handling, and initial request assessment.
  3. Clear separation of concerns between Mailers (for sending) and Mailboxes (for receiving).
  4. Improved email processing efficiency through ActionMailbox, facilitating background job handling and potential simplification of mail ingress.
  5. Provides a solution for re-users to receive emails without needing their own mail server setup.

Integrate ActionMailbox into `RequestMailer.receive` for inbound email
handling. This maintains existing functionality but lays the groundwork
for future enhancements.

Added an `origin` column to the ActionMailbox::InboundEmail table. This
change was necessary to preserve the functionality of checking the mail
`source`, avoiding a naming clash with the `source` method.

Benefits of using ActionMailbox include:

1. Enhanced email routing capabilities, enabling specialized
   processing like the Excel hidden data spreadsheet analyzer.
2. Refactoring opportunities for `RequestMailer#receive`, particularly
   for spam detection, duplicate email handling, and initial request
   assessment.
3. Clear separation of concerns between Mailers (for sending) and
   Mailboxes (for receiving).
4. Improved email processing efficiency through ActionMailbox,
   facilitating background job handling and potential simplification
   of mail ingress.
5. Provides a solution for re-users to receive emails without needing
   their own mail server setup.

WIP
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