Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vertaisarviointi 1 #2

Closed
LauraImmonen opened this issue Sep 24, 2024 · 1 comment
Closed

vertaisarviointi 1 #2

LauraImmonen opened this issue Sep 24, 2024 · 1 comment

Comments

@LauraImmonen
Copy link

Sovelluksen auki saamiseksi oleviin ohjeisiin voisi lisätä, miten ensin tulisi kloonata repo ja tehdä sille hakemisto.

Sovellus näyttää todella hienolta ja oli helppokäyttöinen. Se oli esteettisesti myös mielestäni hienosti tehty. Huomaa että olet tehnyt sovelluksia aikaisemminkin. Koodi oli helppolukuista ja näytti ammattimaiselta.

Sovelluksessa huomasin kohdassa "add exercise" napin "presaved exercises" joka ei tehnyt mitään, sen voisi korjata tai poistaa. Myös kohdassa, jossa voi lisätä "work out result" voisi olla vaihtoehtona kirjata, kuinka monta rep ja set sai tehtyä, koska jos tekee esim. kehonpainoharjoituksia, ei tule käytettyä tuota kg kohtaa. Muuten sovellus oli mielestäni jo toimiva ja hyvin tehty.

@hurbios
Copy link
Owner

hurbios commented Sep 26, 2024

Kiitos hyvästä arvioinnista. Tein muutoksia tämän perusteella ja issuen tulevaisuutta varten kehonpainoharjoituksien tukemiseksi.

Lisäsin ohjeet repon kloonausta varten ja poistin ylimääräisen presaved exercises valikon.
(presaved exercises oli dropdown menu jonka valinta antoi automaattisesti nimen inputtiin, muttei kuitenkaan sarjamääriä (sets) eikä toistomääriä (reps). Näin ollen tuon hyöty jäi vähäiseksi ja tosiaan varmaan parempi poistaa toiminnallisuus.)

Ehdotus muokata sovellusta tulosten osalta niin että tulos voisi olla painon lisäksi myös vaihtoehtoisesti sarjamäärä ja toistomäärä kuullostaa hyvältä. Tämä voisinkin olla käytön kannalta varsin järkevää erilaisten käyttötapausten kattamiseksi. Koen kuitenkin kurssin tähtäävän MVP sovellukseen ja python flask & postgresql yhdistelmän osaamisen osoittamiseen joten tämä ominaisuus kuullostaisikin oikein hyvältä lisätä mahdolliseen kurssin jälkeiseen versioon 2.

muutokset: #3
tulevaisuuden issue kehonpainoharjoittelun tukemiseksi: #4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants