Closed samoslavik45 closed 9 months ago
Nie som si istý, či to bude správne fungovať v takej forme akej to momentálne je tak to prosím pozrite a podľa spätnej väzby to môžem upraviť, ďakujem.
Ten user type v stringu treba previest na UserType enum a tiez zamedzit, aby tam preslo cokolvek ine ako externista/student V modeli
User
nie jeuser_type
povoleny pre mass assignment, treba ho priradit az po create ako$user->user_type = ...;
Tieto veci by mali byť fixnuté.
Čo sa týka funkcie login() neviem teda presne ktorý view by mala zobrazovať alebo ako píše @Mcibula chceme vôbec túto metódu?
Čo sa týka funkcie invite() keďže by mala plniť funkcionalitu z #75 tak môžeme kľudne tú issue priradiť sem a implementujem to.
Problém mám trocha s tým, že by som potreboval objasniť ako získavame údaje z prítomnosti, predstavu ako by fungovalo to pridanie natiahnutých údajov do našej databázy mám ale neni mi úplne jasné, ako k nim pristupovať
Problém mám trocha s tým, že by som potreboval objasniť ako získavame údaje z prítomnosti, predstavu ako by fungovalo to pridanie natiahnutých údajov do našej databázy mám ale neni mi úplne jasné, ako k nim pristupovať
@ihorbaltsat na to robil funkciu v pritomnost connectore. Na synchronizovanie pouzivatelov. Niekto sa ti chce prihlasit, ty pozries, aky je to typ pouzivatela, ak je to zamestnanec alebo doktorand, pouzijes funkciu syncSingleUser (ak si dobre pamatam jej nazov) a az nasledne by si mal kontrolovat spravnost hesla a tak. Ja si ale myslim, ze najprv by bolo fajn implementovat obycajne prihlasovanie, to nech sa mergne a potom sa dokoncia veci s Pritomnostou.
Problém mám trocha s tým, že by som potreboval objasniť ako získavame údaje z prítomnosti, predstavu ako by fungovalo to pridanie natiahnutých údajov do našej databázy mám ale neni mi úplne jasné, ako k nim pristupovať
@ihorbaltsat na to robil funkciu v pritomnost connectore. Na synchronizovanie pouzivatelov. Niekto sa ti chce prihlasit, ty pozries, aky je to typ pouzivatela, ak je to zamestnanec alebo doktorand, pouzijes funkciu syncSingleUser (ak si dobre pamatam jej nazov) a az nasledne by si mal kontrolovat spravnost hesla a tak. Ja si ale myslim, ze najprv by bolo fajn implementovat obycajne prihlasovanie, to nech sa mergne a potom sa dokoncia veci s Pritomnostou.
Hej, ten SynchronizationController
by sa mal dat normalne volat a on by mal za teba spravit tie veci.
Čo sa týka funkcie invite() keďže by mala plniť funkcionalitu z #75 tak môžeme kľudne tú issue priradiť sem a implementujem to.
Dobre, tak ti ju priradim, ked uz to robis, aby sme to urychlili a mozes to spravit aj v tomto PR. Detaily by mali byt tam, ak by ti nieco nebolo jasne, tak sa kludne pytaj.
Po novom funkcia authenticate() by mala synchronizovať údaje zamestnancov alebo doktorandov s prítomnosťou pomocou SynchronizationController::syncSingleUser.
Funkciu login() som teda zmazal.
Funkcia invite() teraz generuje linky ktoré sú časovo obmedzené, vytvára záznam v databáze s použitím modelu InvitationLink a migrácie. Mail sa odosiela pomocou už dávnejšie vytvorenej funkcie SimpleMail.
Použitie invite() som ručne testoval a mail s linkom som úspešne obdržal, záznam sa taktiež uložil do databázy. Jediné čo treba ešte dorobiť aby sa po kliknutí na odkaz presmeroval používateľ na registračný formulár, to som si neni úplne istý na ktorý view to má odkazovať alebo ako presne by to malo byť a taktiež ešte upraviť blade.php ktorý posielame na mail aby to možno vyzeralo trocha krajšie.
Po novom funkcia authenticate() by mala synchronizovať údaje zamestnancov alebo doktorandov s prítomnosťou pomocou SynchronizationController::syncSingleUser.
Funkciu login() som teda zmazal.
Funkcia invite() teraz generuje linky ktoré sú časovo obmedzené, vytvára záznam v databáze s použitím modelu InvitationLink a migrácie. Mail sa odosiela pomocou už dávnejšie vytvorenej funkcie SimpleMail.
Použitie invite() som ručne testoval a mail s linkom som úspešne obdržal, záznam sa taktiež uložil do databázy. Jediné čo treba ešte dorobiť aby sa po kliknutí na odkaz presmeroval používateľ na registračný formulár, to som si neni úplne istý na ktorý view to má odkazovať alebo ako presne by to malo byť a taktiež ešte upraviť blade.php ktorý posielame na mail aby to možno vyzeralo trocha krajšie.
Super, idem to pozriet, vdaka
Este nejake veci treba upravit, ale super praca
Ďakujem za review, dnes to opravím
Este veci, ktore bude treba pridat:
->with('message', ...)
, ze tymto mailovym adresam nebol poslany link, lebo su uz v DB.
Zatiaľ len draft PR, základná verzia user-controlleru.