Skip to content

Conversation

riasvdv
Copy link

@riasvdv riasvdv commented Jul 11, 2024

This still needs an update to Ray to display the attachments (and download them).

@freekmurze
Copy link
Member

Very nice!

@sebastienhenau Could you add support for this in the Ray app?

'filename' => $attachmentPart->getFilename(),
'content_id' => $attachmentPart->getContentId(),
'content_type' => $attachmentPart->getContentType(),
'content' => base64_encode($attachmentPart->getContent()),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it worth checking the size of the content before doing this? Ray can sometimes struggle a bit with large payloads in my experience.

return $this->markdown('mails.test');
return $this->markdown('mails.test')
->attachData('file1', 'file_1.txt')
->attachData('file2', 'file_2.txt');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to test attaching binary files?

@spatie-bot
Copy link

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.

@spatie-bot spatie-bot closed this Nov 20, 2024
@riasvdv riasvdv reopened this Nov 20, 2024
@spatie-bot
Copy link

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.

@spatie-bot spatie-bot closed this Mar 21, 2025
@riasvdv riasvdv reopened this Mar 24, 2025
@spatie-bot
Copy link

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.

@spatie-bot spatie-bot closed this Jul 25, 2025
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.

4 participants