Skip to content

Comments

frida api project #38

Open
fridascript wants to merge 7 commits intoTechnigo:masterfrom
fridascript:master
Open

frida api project #38
fridascript wants to merge 7 commits intoTechnigo:masterfrom
fridascript:master

Conversation

@fridascript
Copy link

@fridascript fridascript commented Feb 6, 2026

Please include your Render link here: https://js-project-api-7sb0.onrender.com/

Frontend link here: https://happy-thoughts-app-frida.netlify.app/

@marinalendt-png
Copy link

Hej Frida!
Grymt jobb med API-projektet <3 Det funkade fint att signup och login. Gillar verkligen login meddelandet uppe till vänster, där det står vem man är inloggad som. Superfint!
Du uppfyller alla kraven för projektet, med dina endpoints, mongoose modeller, lösenordshantering med b-crypt samt fungerande login och signup. Super!
Ett tips, som jag själv tycker är hjälpsamt, är att skapa fler filer och foldrar. Just nu har du det mesta av din kod i server.js, det skulle vara mer överskådligt när man öppnar projektet, att tex ha separata filer frör endpointsen, modellerna mm. Smaksak kanske :)
Jag ser att du har endpointsen för patch och delete, vilket är kanon, men när jag öppnar appen, ser jag inte var jag kan deleta eller edita? Annars funkar ua att skapa en ny tanke och likea den.

Heja dig! Kram från mig <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants