Error when migrating about laravel-starter

[nasirkhan]

Can you please check that the version of MariaDb you are using supports JSON column?

[finnie2006]

I have the same problem, i run mariadb 10.5.13

[finnie2006]

@nasirkhan

[nasirkhan]

I assume the issue is with the MariaDb version. Can you please check if you are using the latest version of MySQL or MariaDb and the JSON column is supported there. I use this starter for my own projects as well and I did not face such issues.

…

-- *Nasir Khan Saikat* www.nasirkhn.com

On Thu, 18 Nov 2021 at 02:51, finnie2006 ***@***.***> wrote: @nasirkhan <https://github.com/nasirkhan> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#292 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADA5OZY3C5KSX3WGCNY3LLUMQIVFANCNFSM5IFQ4VGQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[finnie2006]

With mysql it works not with the latest mariadb

[nasirkhan]

Well, then I think you may need to update the MariaDb version. MariaDb do supports the JSON column in some other latest versions. But if you are using an older version then it might not work as expected.

…

-- *Nasir Khan Saikat* www.nasirkhn.com

On Thu, 18 Nov 2021 at 15:29, finnie2006 ***@***.***> wrote: With mysql it works not with the latest mariadb — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#292 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADA5O3ZYYIRO7LWJFTVNCTUMTBPLANCNFSM5IFQ4VGQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[nasirkhan]

I pushed an update and you should install without any issues now.

…

-- *Nasir Khan Saikat* www.nasirkhn.com

On Thu, 18 Nov 2021 at 16:58, finnie2006 ***@***.***> wrote: 10.5.13 is the latest — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#292 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADA5O4DLWK4SIMFVKWAVI3UMTL5PANCNFSM5IFQ4VGQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[JohnB17]

Fixed. Although, for some reason when I set something in the settings, it doesn't apply. It remembers what I set it to, but it doesn't change anything.

[nasirkhan]

@JohnB17 well you may need to clear the cache to see the changed data. As the Settings Items usually not changed that frequently, so they are cached for a longer time.

[finnie2006]

oh oki, also i tried oauth github but i get the following error when clicking on the button

SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'name' cannot be null (SQL: insert into `users` (`first_name`, `last_name`, `name`, `email`, `updated_at`, `created_at`) values (, , ?, [email protected], 2021-11-20 17:19:18, 2021-11-20 17:19:18)) 

[nasirkhan]

I can see from the query that the First name and Last name is empty so the Name is empty. I always found that Github returns name and email while social login. You may have not set your name on your github account. Please Check the social login with an account with a name.

If you want to allow User accounts without a first name or last name you need to implement that by your own. Or you can handle this exception and show a input form if the social account does not return a Name. If you can build a solution to address this issue, please consider submitting a PR to this repo.