GithubHelp home page GithubHelp logo

nanibgal's People

Watchers

 avatar

nanibgal's Issues

Koodikatselmointi

Projekti ladattu sunnuntaina 7.10. klo 12.30.

Ensivaikutelma oli varsin hyvä. Sovelluksesi näyttää ammattimaiselta - olet käyttänyt hienosti html-elementtejä, projektissasi on paljon tietokantoja ja niiden välisiä yhteyksiä ja sovelluksessa on paljon hyödyllisiä toimintoja. Mahtavaa, että sovellusta voi käyttää sekä englanniksi että suomeksi.

Sovelluksen kansiorakenne on hyvin erilainen kuin kurssimateriaalissa, missä ei sinänsä ole mitään vikaa. Sovelluksen tutkiminen olisi ehkä käynyt helpommin, jos aina yhteen kokonaisuuteen liittyvät tiedostot olisivat olleet samassa kansiossa. Nyt esim. kaikki html-sivut ovat kyllä saman kansion alla, mutta niitä on paljon, joten kokonaisuuden hahmottaminen voisi olla helpompaa selkeämmällä ja yksityiskohtaisemmalla kansiorakenteella.

Sovelluksen käytettävyyttä voisi vielä parantaa. Esimerkiksi haku toimii hyvin, mutta on ehkä vähän epälooginen. Voisiko käyttäjähaku ja viestihaku olla selkeästi omilla sivuillaan, eikä niin, että käyttäjähaun linkki löytyy viestihaun alta? Tai voisiko esimerkiksi haku-linkin alla olla valinta koskien sitä, mitä käyttäjä haluaa hakea ja valinnan mukaan sovellus ohjaa sitten joko viesti- tai käyttäjähakuun?

Aloitussivua voisi ehkä myös miettiä. Kirjautumissivu sinällään on ok aloitussivuna (etenkin jos kaikki sovelluksen toiminnot vaativat kirjautumisen), mutta rekisteröitymiseen voisi olla sujuvampi yhteys. Sovelluksessa voisi olla esimerkiksi selkeämpi linkki rekisteröitymissivuun, jotta sen huomaa heti eikä sitä joudu etsimään kirjautumissivun pohjalta.

Viestit toimivat hyvin. Tuli vielä mieleen, että jos johonkin viestiin vastaa ja sen jälkeen alkuperäisen viestin poistaa, vastaukset jäävät silti elämään. En tiedä, onko tämä välttämättä huono asia, mutta käyttäjän kannalta on vähän hassua nähdä pelkät vastaukset, mutta ei sitä, mihin vastataan.

En löytänyt repositoriostasi valmista testikäyttäjää (mikä voisi olla kätevä sovelluksen testaamiseen), mutta tein itse uuden. En päässyt testaamaan muita kuin normaalikäyttäjän oikeuksia, joten valvojan user storyjen toimivuudesta en osaa sanoa mitään. Kaikki muut user storyt mielestäni toimivat paitsi käyttäjän seuraaminen. Loin siis kaksi käyttäjää ja yritin seurata toista käyttäjää, mutta follow-toiminto aiheuttaakin internal server errorin. Tällöin myöskään feedissä en saa mitään muuta näkyviin kuin itseni lähettämät viestit. Tämä lienee hyvä korjata seuraavaksi?

Projektissasi on paljon koodia, josta iso osa on sellaista, mitä en oikein ymmärrä ja sen läpikäyminen ajatuksella veisi turhan paljon aikaa. Yleisilme on kuitenkin hyvä: metodien ym. nimeäminen on järkevää ja näyttää omaan silmääni asianmukaiselta. Olet myös saanut paljon aikaan todella lyhyessä ajassa ja korjattavaakin löytyi vain hieman, hyvää työtä.

Tsohan koodikatselmointi

Latasin projektin koneelleni 18.10.2018.

Kokeilin sitä paikallisesti sekä herokun kautta. Seuraavia asioita huomasin, kun kokeilin sovelluksen toimintaa:

  • Sovelluksen käyttö oli helppoa ja luontevaa. Sen käyttämisen oppi hyvin nopeasti!
  • Ulkoasu oli todella tyylikäs. Värit valkoinen, vaaleansininen ja harmaa sopivat hyvin toisiinsa.
  • Sivutus oli näppärä, kun postauksia oli paljon
  • Työ vaikutti aika laajalle, mutta hienosti olit saanut sen toimimaan näinkin lyhyessä ajassa
  • AChecker ei löytänyt mitään valitettavaa html sivuistasi, enkä löytänyt itsekään
  • Siisti idea tuo, että sovellusta pystyi käyttämään kahdella eri kielellä
  • Kun yritin herokussa poistaa käyttäjän tuli vastaan Internal Server Error. Jos yritin poistaa käyttäjäni, mutta jätin käyttäjänimen pois, päädyin jostain syystä ilmoituksiini. Jos yritin poistaa käyttäjäni, mutta kirjoitin käyttäjänimen väärin, ei tapahtunut mitään, mutta en saanut virheilmoitusta. Myöskään lokaalisti ei käyttäjän poistaminen onnistunut. Ehkä hiot sitä vielä.
  • Kun klikkasin sen jälkeen omaa käyttäjänimeäni, näkyi välillä tykkäyksiä ja vastauksia viestissä enemmän (molempia 4) kuin olin niitä sinne laitettu (molempia 2). Kun klikkasin itse postauksen auki tai jos yritin vastata omaan postaukseen oli tykkäyksiä ja vastauksia oikea määrä (molempia 2). Joku bugi varmaan siinä tykkäyksien ja vastauksien määrän näyttämisessä? Toisella käyttäjällä näkyivät tykkäyksien ja vastausten määrät koko ajan oikein.
  • Koodisi oli selkeää ja ymmärrettävää!

Muutama jatkokehitysidea tuli mieleeni, jos innostut vielä kehittämään sovellusta:

  • Olisi kiva jos vanhat notifikaatiot löytyisivät jostain :)

Dokumentaatio oli tehty hyvin ja vaikutti hyvin valmiilta! Hieno sovellus kaiken kaikkiaan.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.