search

calmdevelopment / obi-wan-kinobi

Filmpflege dingens für kleine Programmkinos
0 stars 0 forks source link

Provide standard auth #3

Closed tpraxl closed 5 years ago

tpraxl commented 5 years ago

Das ist einfach nur das Ergebnis von php artisan make:auth. Es erstellt ein paar basic forms und die Auth-Routes. Muss noch angepasst werden, damit sich nicht einfach jeder für das Backend registrieren kann.

Wenn noch nicht geschehen, müsste nach dem Checkout mal ein php artisan migrate ausgeführt werden, damit die DB-Tabellen angelegt werden.

commit message

Notice: Every user can register without confirmation at this point. This is just the standard out-of-the-box laravel auth setup.

We need to add a confirmation field to prevent arbitrary users from creating functioning accounts.

ancho commented 5 years ago

Ok soweit. Ist das dann noch WIP, oder kann ich das mergen?

tpraxl commented 5 years ago

Kann gemergt werden. Wenn ich es nicht extra schreibe sind alle Pull-Requests zum Mergen gedacht. Für andere Fälle hat man ja einfach Branches, auf die man verweisen kann. Warum werden die Branches eigentlich nicht automatisch gelöscht, wenn sie gemergt werden? Hm.. das ist in gitlab glaub ich besser gelöst.

ancho commented 5 years ago

Hmm. Gute Frage. Ich bin mir gerade nicht ganz sicher. Aber ich meine ich habe auch hier schon angeboten bekommen den branch zu löschen.