Open neononoen opened 7 months ago
Thanks for the feedback! The finnish messages are there temporarily so that I can test psql connection. They'll definitely be gone by next return. It's true that README.md
could give a better overview of the and also the UI could get faster. I'll try to make the code more consistent, maybe autopep8 it. Cheers
Vertaisarviointi
Sovelluksen idea kuulostaa kuvauksen perusteella mielenkiintoiselta vaikkakin haastavalta. READMEstä ei käynyt ilmi, mikä sovelluksen nykytilanne on, joten en ole varma mitä kaikkia toimintoja sovelluksessa tällä hetkellä pitäisi olla. Ainakin rekisteröityminen ja sisäänkirjautuminen toimii sekä viestin lisääminen etusivulle. Sisääkirjauduttua pystyi lataamaan pdf-tiedostoja ja lisäämään kirjoja pariksi. Omia latauksia ja pareja pystyi myös tarkastella. Muut toiminnot eivät ilmeisesti vielä toimi.
Huomioita sovelluksesta: Sovelluksen käyttöliittymä oli muuten englanniksi, mutta viestien määrä jostain syystä luki suomeksi. Onko viestien lisääminen etusivulle ylipäätänsä tarpeellista tai minulle jäi hieman epäselväksi mikä niiden idea on? Kirjoja tai pareja ei ollut mahdollista poistaa, joten se saattaisi olla hyvä lisä sovellukseen. Kirjan lisäämisen jälkeen sovellus voisi myös palata takaisin etusivulle, sillä nyt joutuu jonkun verran klikkailemaan taaksepäin.
Sovelluksen koodi vaikuttu hyvältä. Jossain kohtaa koodin yhdenmukaisuuteen voisi kiinnittää huomiota esim. heittomerkit.