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] Added dateTime to columns that don't need character options #39269

Merged
merged 1 commit into from
Oct 20, 2021
Merged

[8.x] Added dateTime to columns that don't need character options #39269

merged 1 commit into from
Oct 20, 2021

Conversation

Dakoni4400
Copy link
Contributor

I stumbled across an error with altering some dateTime columns that led me to #30539. I was changing a string column to a dateTime column. DateTime is missing from the list of columns that don't need character options so I added it in, tested it in my application and it worked just fine. So that PR adds dateTime to the list of columns that don't need character options

@Dakoni4400 Dakoni4400 changed the title Added dateTime to columns that don't need character options [8.x] Added dateTime to columns that don't need character options Oct 20, 2021
@taylorotwell taylorotwell merged commit 2c7260d into laravel:8.x Oct 20, 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.

2 participants