Henrik S, Sarah T & Nathalie L#19
Conversation
Removed test lines and updated content structure.
Reformatted text for better readability by breaking long sentences into shorter ones.
Updated formatting for the section on trust boundaries.
Added HTML break tag for formatting in the document.
Fixed HTML break tags in the text about trust boundaries.
Updated security risks and requirements in the planning phase document, including detailed scenarios for identified threats and formulated security requirements to mitigate them.
Removed unnecessary explanations and streamlined security requirements section.
Co-authored-by: nat316 <nat316@users.noreply.github.com> Co-authored-by: HS-devs <HS-devs@users.noreply.github.com>
Co-authored-by: nat316 <nat316@users.noreply.github.com> Co-authored-by: satj-lab <satj-lab@users.noreply.github.com>
Co-authored-by: HS-devs <HS-devs@users.noreply.github.com> Co-authored-by: satj-lab <satj-lab@users.noreply.github.com>
isomoth
left a comment
isomoth
left a comment
There was a problem hiding this comment.
Bra jobbat! Man förstår hur ni tänkte och organiserade arbetet. Jag skulle evtl. flytta de långa kommentarerna till en separat fil, men det är bara en lite detalj.
| JWT_SECRET=your-secret-here | ||
| FRONTEND_URL=http://localhost:5500 | ||
|
|
||
| # // FRONTEND_URL=http://localhost:5500 Det är en okrypterad adress och inte säker att använda. Är detta endast för utveckling? Säkerställ att det inte används i produktion. No newline at end of file |
There was a problem hiding this comment.
Tomma rader blir rödmarkerade för att de kan orsaka problem på vissa system. Ett linter-verktyg såsom Prettier (VS Code Extension) brukar rensa bort dessa små detaljer :)
| @@ -1 +1,37 @@ | |||
| # yh-message-app-fullstack No newline at end of file | |||
| # yh-message-app-fullstack | |||
There was a problem hiding this comment.
Jättebra
med den här README, det vissar er arbetsprocess på ett bra sätt.
| @@ -1 +1,68 @@ | |||
| # Inlämning 1 - Planeringsfasen No newline at end of file | |||
| # Inlämning 1 - Planeringsfasen | |||
There was a problem hiding this comment.
Kort och koncis beskrivning på er planeringsfas, nice!
There was a problem hiding this comment.
Det märks tydligt att ni har försökt dokumentera era tankegångar och säkerhetsresonemang i koden, vilket gör det lättare att förstå syftet bakom vissa lösningar och modifieringar. Det känns särskilt värdefullt i lärandesammanhang där fokus ligger på att visa förståelse för processen.
En möjlig förbättring hade dock kunnat vara att dela upp vissa längre kommentarer och koppla dem mer direkt till specifika kodrader eller kodblock. Det hade gjort kommentarerna ännu lättare att läsa och följa i relation till den faktiska implementationen.
✅ Deploy Preview for yh-message-app-fullstack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Clarified the explanation of backend permission checks and improved wording for better readability.
6 participants
No description provided.