-
Notifications
You must be signed in to change notification settings - Fork 257
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
Username field not created by confide:migration #509
Comments
Hi @iforwms-snippets 'username' => $emailOrUsername line 103 in file vendor/zizaco/confide/src/Confide/EloquentRepository.php |
Same problem. As EloquentRepository.php is part of the core package files, I've just added username back into the migration file (replacing ->unique() with ->nullable()). |
Hi, Try using the --username option. This will add the username field to the migration. php artisan confide:migration --username Also don't forget to use that same --username option for creating the controller: php artisan confide:controller --username One immediate problem that occurs is that the username field isn't a required field, but it should be unique according to the database schema. Thus if on the registration form the username isn't entered and there already is a record stored with an empty username, a QueryException (Integrity constraint violation) is given. |
Gonna fix this asap |
Same here :( |
Status please. Currently setting up my login. Dont want to use the username or atleast have the possibility to choose. |
I know this is specified in the docs as not being created (It will setup a table containing email, password, remember_token, confirmation_code and confirmed columns, which are the default fields needed for Confide use. Feel free to add more columns to the table later.)
But the sql query checks for username OR email equal to the customer input. As the username column does not exist by default, this throws an error.
The text was updated successfully, but these errors were encountered: