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

Open
risla763 opened this issue Sep 28, 2023 · 0 comments
Open

vertaisarviointi #1

risla763 opened this issue Sep 28, 2023 · 0 comments

Comments

@risla763
Copy link

Hyvä alku!

Ohjelma aukesi selaimeen ja siinä on aloitussivu, josta pääsee nappia painamalla kirjautumissivulle. Tämä on hyvä, että sivusta pääsee toiseen painamalla nappia.
Aloitussivulle on jopa liitetty hieno kuva 👍 .

Kirjautumissivulla on tekstikentät joihin kirjoitetaan käyttäjänimi ja salasana niin ne toimivat muuten hyvin, mutta salasana kohdassa salasana olisi hyvä näkyä käyttäjälle "piilotettuna", esimerkiksi näin: "moi" = "***".

Myös kannattaa tehdä "/logout" route kansioon app.py, millä käyttäjä voi kirjautua ulos.

Koodi siistiä, kuitenkin routes metodien kannattaisi olla eri nimisessä tiedostossa kuin app.py, koska app.py tiedostossa yleensä on vain app = Flask(name), sekä ohjelman käyttämän tietokannan polku yms. Nämä "reitit" eli "routes" kannattaa siirtää omaan tiedostoon nimeltä esimerkiksi routes.py. Kannattaa katsoa esimerkkisovelluksesta mallia.
Kannattaa lisätä app.py tiedoston loppuun import routes.

Hyvä että aloitussivun kuvan tiedosto on omassa "static" nimisessä kansiossa.

HTML tiedostot myös siistejä. Ehkä kannattaa lisästä hieman kommentteja, jotka kertoisivat mitä eri kohdat koodissa tekevät.

Kannattaa lisätä navigointipalkki. Siis palkki vaikka sovelluksen yläreunaan, jossa on ns. eri sivuille johtavat napit, että voit navigoida sovelluksessa helposti.

Kuitenkin hyvässä alussa oleva sovellus ja selkeää koodia. Jatka samaan malliin 😄

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

1 participant