theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Separate navbars #75

Closed ITJohan closed 4 years ago

ITJohan commented 4 years ago

Navbaren visar endast login när man är utloggad, och trips, account och logout när man är inloggad. Fixade också så att inloggad state ändras när man registrerar eller loggar in. Klickar man logout så loggas man ut.

Logout verkar fungera men ger GET http://spilg.xyz/api/login/ net::ERR_ABORTED 500 (Internal Server Error). Man kan logga in utan att skriva in något men det är ett problem med Login.js och behandlas inte i denna PR.

Edit: Ändrade även till http i config.

AndreasWJ commented 4 years ago

Tjenare! Har börjat kolla igenom koden. Ser bra ut för övrigt. Men tänkte om det fanns ett sätt att hämta information från sessionen, jag tänker framförallt på userID. Med userID kan vi sedan /api/users/:userID/ och hämta information såsom namn. Vilket kan användas för att få upp en "profilbild" i navbaren med användaren initialer.

Ska se om jag hittar något. Med JWTs kan man ju läsa, så bör ju finnas nånting med sessioner också.

ITJohan commented 4 years ago

Tjenare! Har börjat kolla igenom koden. Ser bra ut för övrigt. Men tänkte om det fanns ett sätt att hämta information från sessionen, jag tänker framförallt på userID. Med userID kan vi sedan /api/users/:userID/ och hämta information såsom namn. Vilket kan användas för att få upp en "profilbild" i navbaren med användaren initialer.

Ska se om jag hittar något. Med JWTs kan man ju läsa, så bör ju finnas nånting med sessioner också.

Japp håller med, har haft svårt att jobba på grund av alla backend-problem och oklarheter med cookies. Därför blev det denna version. :) Det verkar som att vi kommer få tillgång till en token/sessionID som vi kan använda för att identifiera oss till backend. Då kan jag bara byta allt till token och setToken eller liknande för samma funktionalitet.

ITJohan commented 4 years ago

@AndreasWJ Fixade så att att vi har en "global" loggedInUser nu med token och email enligt det nya backend-systemet. Ge gärna feedback på vad du tror om det. :)

Edit: Vet inte om den fungerar 100% eftersom nya backend inte har mergats än, men tänker mig att det kommer fungera ungefär så här.

ITJohan commented 4 years ago

Det har du helt rätt i, vet inte varför jag skicka med det i Form :sweat_smile: Slänger upp en fix!

AndreasWJ commented 4 years ago

Det har du helt rätt i, vet inte varför jag skicka med det i Form 😅 Slänger upp en fix!

Tagga mig här när den är uppe! Väntar med att göra ändringar i min branch, så jag slipper konflikter

ITJohan commented 4 years ago

@AndreasWJ Så nu ska det mesta med logged in state och requests vara fixat. Löste några problem som fanns i den andra PRn som mergades också.