Understrukna ord finns i lektionsordlistan. Tryck på ett ord för att se alla översättningar. Lägg till eller redigera ord i messages/glossary.json (alla språknycklar du lägger till visas i rutan).
Coding, debugging, and design
Koden ser jättebra ut överlag, bra jobbat!
Jag har lämnat några mindre kommentarer i din pull request.
Det här variabelnamnet kunde kanske vara lite mer beskrivande för vad det faktiskt gör.
Har du funderat på att bryta ut den här stora funktionen i mindre, separata metoder?
Jag tror vi kan förenkla den här logiken med en inbyggd metod istället.
Koden är väldigt ren och lättläst, det gillar jag skarpt.
Glöm inte att ta bort dina utkommenterade rader och console.logs innan vi mergar.
Vi bör nog lägga till ett enhetstest just för den här specifika kanten (edge case).
Skulle du kunna lägga till en kort kommentar som förklarar varför vi gör på det här sättet?
Det ser ut som att den här variabeln deklareras men sedan aldrig används någonstans.
Jag är lite orolig för prestandan i den här nästlade loopen, kan vi optimera den?
Kan du snabbt verifiera att koden följer vår etablerade kodstandard (linting)?
Vi kanske borde använda en konstant istället för att hårdkoda 'magic strings' här i filen.
Den här filen har blivit ganska stor, kanske är det dags att refaktorera lite?
Jag godkänner (approvar) pull requesten, du kan merga när byggena lyser grönt.
Är det möjligt att vi oavsiktligt introducerar en minnesläcka här om vi inte stänger strömmen?
Riktigt elegant och snygg lösning på ett annars ganska komplicerat problem!
Jag begär tyvärr några ändringar innan jag känner mig trygg med att godkänna.
Ska vi boka ett kort möte så att du kan förklara tanken bakom den här logiken?
Det ser ut som att vi duplicerar väldigt mycket kod från en annan fil här.
Kan vi använda valideringsbiblioteket vi redan har istället för att bygga egen validering?
Har du kört alla tester lokalt för att se att inget annat i systemet har gått sönder?
Var uppmärksam på att den här funktionen numera är markerad som 'deprecated' i ramverket.
Bra jobbat med att städa upp bland all den gamla och föråldrade koden (legacy-koden)!
Jag gillar verkligen hur du strukturerade om mapparna, det blev mycket tydligare nu.
Glöm inte att vi också måste uppdatera den tekniska dokumentationen i README-filen.
Det finns risk för ett null-pointer exception om 'data'-objektet inte returneras av API:et.
Kunde vi hantera felen mer strukturerat med en ordentlig try-catch-block här?
Jag tycker vi bör undvika att mutera det ursprungliga objektet direkt i funktionen.
Borde vi inte döpa om den här filen så att namnet matchar den nya funktionen?
Logiken ser helt korrekt ut, men vi kan formatera koden lite snyggare.
Kan vi lägga till mer kontext i felmeddelandet så att vi enklare kan felsöka i framtiden?
Var försiktig, den här databasfrågan riskerar att hämta ner alldeles för mycket data.
Jag rekommenderar att vi sätter en tydlig timeout på det här nätverksanropet.
Snygg refaktorering, koden är betydligt mycket mer modulär nu!
Ser ut som en bra lösning, men kolla så att den fungerar i äldre webbläsare också.
Jag ser att du har använt var, vi brukar vanligtvis använda const eller let numera.
Det känns som att den här komponenten gör lite för mycket saker samtidigt.
Jag vill bara dubbelkolla: är det meningen att den här funktionen ska vara asynkron?
Tänk på att vi bör undvika onödiga renderingar i den här React-komponenten.
Ett tips: du kan använda 'early return' för att undvika djupa nästlingar i koden.
Kan du uppdatera beroendet i package.json så vi får med säkerhetsuppdateringen?
Vi saknar felhantering för när databasen eventuellt är otillgänglig.
Bra att du lade till try/catch, men vi borde också logga själva felet till systemet.
Koden är godkänd från min sida, men be gärna designern titta på gränssnittet också.
Är du helt säker på att typhanteringen (TypeScript) stämmer överens med API-svaret?
Vi måste säkerställa att vi inte råkar logga ut känslig användardata i konsolen.
Jag älskar det nya mönstret du introducerade, låt oss använda det i framtiden.
Kan du ta en extra titt på om vi kan spara lite resurser på databasservern här?
Jag är supernöjd med din kod, skicka in den i produktion när du känner dig redo!
