search

eporsche / daybreak

Simple timesheets and vacation management for small businesses.
MIT License
127 stars 37 forks source link

[BUG] - User DB seeder #12

Closed Nejcc closed 3 years ago

Nejcc commented 3 years ago

There is a problem with user DB seeder when you change email and password inside DatabaseSeeder

app(CreatesNewUsers::class)->create([
            'name' => 'Admin User',
            'email' => 'admin@daybreak.corp',
            'password' => 'admin1234',
            'password_confirmation' => 'admin1234',
            'terms' => Jetstream::hasTermsAndPrivacyPolicyFeature() ? true : false,
        ]);
FelixBoers commented 3 years ago

Hi Nejcc,

Thanks for your interest in this project. What exactly is your problem here? I changed the email and password to something else and then executed ./vendor/bin/sail artisan db:seed without any problems.

However, I noticed two things that might help you further:

  1. Verify that password and password_confirmation are equal.
  2. Verify that there is no other user registered with the same email.
nejcdev commented 3 years ago

Why you dont use bcrypt('admin1234') on password ?

Nejcc commented 3 years ago

@FelixBoers i will try to check i just check or fly trough the project and see some issues

FelixBoers commented 3 years ago

@nejcdev Because then it would be encrypted twice @Nejcc Thanks for your reply. If you still have an issue, please open this ticket again.