search

fjohansson92 / 1DV450_fj222dr

0 stars 1 forks source link

Peer Review 3 - ja222qm #13

Open juhaniaa opened 9 years ago

juhaniaa commented 9 years ago

Kod

Kul att se att du använt grunt! Du kunde gärna haft några kommentarer i readme-filen om hur man kan köra tex. tester osv då du ändå har mycket av detta upplagt. Generellt tycker jag du har en tydlig uppdelning av koden med olika moduler för tex "login" och "restaurants" vilket gör det enkelt att hitta bland filerna. Det som saknas är väl lite kommentarer för att enklare hänga med i vad som händer. Du har till största delen en tydlig namngivning. Som en väldigt liten detalj kunde nämnas att din "restaurant- och user-factory" har stor första bokstav, men inte din "tag-factory". Bra att du har lagt autocomplete funktionalitet i egna direktiv, dock skulle deras templates vara lättare att greppa om de hade sina egena filer.

Live applikation

Jag tycker ditt gränssnitt känns tydligt och genomtänkt. Lätt att förstå vad man kan/ska göra. Bra sökfunktioner med autocomplete och filter samt smidig paginering av listan med restauranger, mycket bra jobbat! Restaurangerna uppdateras automatiskt när man flyger runt på kartan vilket är nice! Lite svårt att lista ut hur det väljs ut vilka som visas, jag blir som användare lite osäker på om jag ser alla restauranger i ett land eller inte då det kan dyka upp flera när jag zoomar in närmare. Extremt enkelt att logga in med github! Dock kan jag inte lägga till någon restaurang utan får ett snabbt meddelande om "Bad credentials" trots att jag är inloggad. Utöver detta fungerar din live applikation utmärkt!

Sammanfattning

Jag tycker att din "Restaurant finder" klient-applikation är välbyggd och utnyttjar ditt bakomliggande API på ett smart sätt. Du har tagit fördel av funktionaliteten i angular för att erbjuda användaren en bra upplevelse. Med några små fixar har du en början till en succe-app! :+1: Bra jobbat!

// Juhani - ja222qm

fjohansson92 commented 9 years ago

Tack för feedback! Försökte göra lite som Google maps med att det laddas in när man flyttar runt. Skillnaden är ju att de har små prickar för de markörer som inte visas, hade ju blivit tydligare då. Som det är nu visas alla men de är paginerade. Problemet var mina custom headers som serven la till HTTP på, men det är fixat så det ska gå att lägga till restauranger nu.