Skip to content
This repository has been archived by the owner on Nov 21, 2022. It is now read-only.

migration file to use longtext instead of text for email_log table #12

Merged
merged 2 commits into from May 11, 2018
Merged

Conversation

hapheus
Copy link
Contributor

@hapheus hapheus commented May 11, 2018

I send a few mails with attached pdf documents.
Unfortunately they can not get saved and are not sent.
The reason is that the MySQL TEXT data type can store only 64kB of data.
MEDIUMTEXT can save 16MB and LONGTEXT can store up to 4GB.
So I simply changed the datatype of attachments from TEXT to LONGTEXT.

@neochief
Copy link
Contributor

Hi!

Thanks a lot for your contribution! Could you please update your migration so that it would use Schema builder instead of the raw SQL queries? See: https://laravel.com/docs/5.6/migrations#columns

The reason I'm asking is that some DB Servers might have the LONGTEXT type called differently, but it supposed to be safely handled by the Schema builder.

@hapheus
Copy link
Contributor Author

hapheus commented May 11, 2018

Hi,

thanks for the hint.
I just updated the migration file.

thanks and have a great day

@neochief neochief merged commit 8710cb8 into shvetsgroup:master May 11, 2018
@neochief
Copy link
Contributor

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants