laravel / ideas

Issues board used for Laravel internals discussions.
940 stars 28 forks source link

Laravel Breeze install overwrites web routes #2544

Closed j-dexx closed 3 years ago

j-dexx commented 3 years ago

https://github.com/laravel/breeze/blob/1.x/src/Console/InstallCommand.php#L78

This behaviour is fine if your steps are new project then install breeze.

On an existing laravel application it's really quite annoying.

michaeldyrynda commented 3 years ago

The idea of the scaffolding is to scaffold a new application. None of the scaffolding (Breeze or Jetstream or even laravel/ui) is intended to be added to an existing app.

j-dexx commented 3 years ago

Ok, I find that a little weird but fair enough. I get they're listed as starter kits, but it'd be nice to have the option to add them to an existing application later on without just copying all the files across manually

susanBuck commented 3 years ago

@j-dexx - I ran into the same frustration. Was expecting it to append routes like laravel/ui does, not wipe the routes file clean.

I found that Fortify is a better fit for incorporating authentication into existing applications.

j-dexx commented 3 years ago

Thanks @susanBuck I'll look at Fortify next time I need to add auth.