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

[8.x] Add tinyText data type to Blueprint and to available database grammars #36949

Merged
merged 3 commits into from Apr 12, 2021
Merged

[8.x] Add tinyText data type to Blueprint and to available database grammars #36949

merged 3 commits into from Apr 12, 2021

Conversation

bennett-treptow
Copy link
Contributor

@bennett-treptow bennett-treptow commented Apr 12, 2021

tinytext is a valid MySQL data type that indicates a 255-byte string field, and is easily stubbed with varchar or text fields for the other DBMS's. tinyInteger is lonely! Currently, tinytext is not implemented for use in Blueprint.

This PR aims to add the tinyText method to Blueprint, with the native tinytext data type being used for MySQL; SQLite will use a text field, Postgres will use a varchar(255) field, and SQL Server will use a nvarchar(255).

This PR only adds new functionality, so it would be backwards compatible since no one would be using tinyText in their migrations.

This could be helpful to those that wanted native support for the tinytext data type in MySQL databases.

@bennett-treptow bennett-treptow changed the title Add tinyText data type to Blueprint and to available database grammars [8.x] Add tinyText data type to Blueprint and to available database grammars Apr 12, 2021
@taylorotwell taylorotwell merged commit d834187 into laravel:8.x Apr 12, 2021
bennett-treptow added a commit to bennett-treptow/docs that referenced this pull request Apr 13, 2021
taylorotwell pushed a commit to laravel/docs that referenced this pull request Apr 13, 2021
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.

None yet

2 participants