Matoo125 / Vegapo

0 stars 2 forks source link

Facebook login. #7

Closed mrkovec closed 7 years ago

mrkovec commented 7 years ago

Moznost pouzit Facebook OAuth pri:

Basic buttony pre oauth funkcionalitu pridane do modalnych okien pre prihlasenie a registraciu a do uzivatelskeho profilu. Dalsi dizajn nacham radsej na profika.

Matoo125 commented 7 years ago

Pozeral som https://hybridauth.github.io/ a vyzerá to dosť dobre. Dá sa tam spraviť Facebook / Google auth spoločne. Možno by to stálo za zváženie.

mrkovec commented 7 years ago

Ze som na to nenarazil skor :) Ten FB oauth mi prisiel zvladnutelny, tak som ani nehladal nejake helpery. Nevadi, ak by bol zaujem o dalsie moznosti prihlasovania (a teda ze ich podporuje pozehnane) nie je problem ich spristupnit pomocou tejto kniznice.

Matoo125 commented 7 years ago

Ok, takže ty si to už spravil 😄 . Vyzerá to super, pokúsim sa spojazdniť.

Jedine ten Google by sa teoreticky hodil, ale nieje to moc dôležité.

Matoo125 commented 7 years ago

Takže aj to funguje dobre.

mrkovec commented 7 years ago

Ešte by som zmenil tlačítko "connect facebook" na "disconnect facebook" alebo by som ho iba schoval pro tých, čo už sú napojený.

Myslel som zeby mozno bolo dobre im to nechat, ak by si chceli connectnut ine fb konto ako uz maju. Ale je fakt, ze kolko uzivatelov by nieco take aj realne spravilo. Mozno najlepsie riesenie by bolo pridat tlacidlo disconnect (aj s tym umozornenim na nezadane heslo).

Možno nejaká poznámka že "ak sa chcete prihlásiť pomocou emailu prosím zadajte si heslo" pre tých čo sú napojený iba na fb

Dobry napad.

Matoo125 commented 7 years ago

Ja by som dal disconnect iba s heslom a potom si môžu znova pripojiť hocijaký účet. Zadať heslo zas tak ťažké nieje a aspoň sa nám tu neobjavia neprihlásitelné účty.

Nechám to na teba, keď tento detail doriešiš spravím merge.

mrkovec commented 7 years ago

Upravene. Skus pozriet.

Matoo125 commented 7 years ago

completed