Katselmointi

• Heroku linkki olisi kiva olla readmessa

Sivuston käytettävyys

Mitä tein

• 28.7.2019 klo ~13:10

• Käytin sovellusta kirjautumatta

  • Kävin välilehdillä ja välilehtien linkeissä

• Rekisteröidyin omalla tunnuksella (jonka hyväksyin administa)

  • Tykkäsin kuvista ja Lisäsin kuvan

• Adminina lisäsin tapahtuman

Kokemus

• Hyvä README.md kuvaus, mukava kuulla myös kirjastoista vaikka hieman keskeneräisyyttä vaikuttaa olevan. Ympäristönpystytysohjeet voisi olla mukana: miten voin kehittää?

• Uutena rekisteröintinä ohjattiin etusivulle jossa edelleen luki "Login or Register to apply membership". Ehkä pitäisi viedä "tiliäsi käsitellään" sivulle.

• Sivun päivittäminen millä tahansa muulla sivulla kuin etusivulla epäonnistuu. En tästä syystä pystynyt bookmarkaamaan sivuja, yritin avata toista välilehteä lukeakseni TOSin mutta sainkin "unknown endpoint".

• Tähän liittyen, favicon olisi hyvä laittaa kuntoon
  

• Events sivulla aloitus- ja loppumisajat voisivat olla yksinkertaisemmassa muodossa.

• Todella responsiivinen tuntuma. Pidin tykkäysten lisäämisestä mediumin "clap" tapaisina vaikka ensin odotin facebook tyylistä tykkäystä.

• Tykkäyksissä mennään yksi alaspäin jos vaihdan välilehteä: Paina tykkää painiketta niin että kuvalla on 10 tykkäystä, vaihda välilehti Artists ja sitten takaisin Gallery, kuvan tykkäykset laskevat 9.

• Kenttien validointi 👍

Koodi

• Kansiorakenne selkeä. Kaikki löytyy juuri sieltä mistä odottaisi.

• Selkeästi nimetyt tiedostot. 👍

• Koodiin jäänyt aika paljon vanhaa koodia kommentoituna. Suosittelen kommenttien sijaan käyttämään versionhallintana gitiä 😉

• Kannattaa erottaa navbar omaksi komponentiksi.

• Navbarin osuuden conditional rendering on selkeä (vaikka itse tiedosto sisältääkin monta asiaa), mutta esim https://github.com/vsvala/Art_Club/blob/4d445dbd0bbfb29f69b3aa28c0694564dbe95f25/src/components/artwork/ArtworkList.js on aika paljon logiikkaa yhden returnin sisällä.

• Conditional renderingistä puheenollen https://github.com/vsvala/Art_Club/blob/4d445dbd0bbfb29f69b3aa28c0694564dbe95f25/src/components/artwork/SingleArtwork.js voisi helposti parantaa siirtämällä logiikkaa returnista pois. https://reactjs.org/docs/conditional-rendering.html

• Backin puolelle voisi siirtää READMEen omat kirjastot

• Back todella selkeä myös, ainoa muutosidea (makuasia) olisi keskittää kaikki routet yhteen tiedostoon ja sisällyttää kontrollereihin itse funktiot selkeästi nimettyinä.

Kokonaisuus

• Todella hienot sivut. Designiin käytetty selkeästi aikaa.

• Suurin harmi oli sivupäivityksen epäonnistuminen, mutta muut ongelmat olikin todella pientä.

• Selkeä elegantti sovellus joka pienellä viilauksella sopisi mobiilikäyttöön.

🎨 / 🖌