Skip to content

Conversation

fmarslan
Copy link

@fmarslan fmarslan commented Apr 17, 2025

Hi!
This PR addresses issue #2141 and includes a feature to forward emails with attachments.
Kindly requesting approval of the workflows and a review when you have a chance.
Thank you!

image

@RitvikSardana
Copy link
Member

Please put a video of this feature

@fmarslan
Copy link
Author

fmarslan commented Apr 18, 2025

helpdesk-forward-ticket

@RitvikSardana
Copy link
Member

Feels unnecessary feature to be honest @fmarslan

We can just use the reply icon and add CCs & BCCs there

@fmarslan
Copy link
Author

Using CC and BCC in the reply is helpful, but forwarding has a different purpose. For example, when we forward an email, we usually want to include attachments and keep the original content without indentation. In replies, attachments are not included automatically and the message gets indented.

So instead of adding these features into the reply function, it makes more sense to have a separate forward feature to handle this properly.

@fmarslan
Copy link
Author

fmarslan commented May 21, 2025

@RitvikSardana

Hi, Just a gentle reminder regarding this PR. It's been open for a while, and I wanted to check if there's anything else needed from my side to move it forward.

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