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

Pregled projekta #1

Open
jaanos opened this issue Jul 8, 2020 · 2 comments
Open

Pregled projekta #1

jaanos opened this issue Jul 8, 2020 · 2 comments

Comments

@jaanos
Copy link

jaanos commented Jul 8, 2020

Pregledal sem vajin projekt - glede na to, da vse deluje, bom dodal povezavo na repozitorij predmeta. Če se še nista, se lahko z @alenFMF domenita za termin zagovora. Če bosta do takrat še kaj spreminjala, mi javita, da posodobim povezavo.

Imam še nekaj komentarjev:

  • Trenutno se v bazi sklicujeta neposredno na uporabniška imena. Bolje bi bilo, če bi za glavni ključ v tabeli uporabnik imela ID (uporabnisko_ime naj bo potem UNIQUE), pa se potem povsod sklicujeta na ta ID.

  • Gesla v bazi so trenutno v čisti obliki - dobro bi jih bilo zakriti (poglejta npr. Fakebook).

  • Pri iskanju dogodkov bi te bilo dobro ustrezno razvrstiti (npr. po datumu).

  • Ko se po iskanju uporabnik pridruži dogodku ali ga zapusti, bi bilo bolje preusmeriti drugam, npr. na stran dogodka ali na prvo stran (da se ne pokaže prazno iskanje).

  • Pri pregledovanju profila drugega uporabnika bi bilo dobro zamenjati Moje objave z Objave (in podobno za aktivnosti).

  • Smiselno bi bilo videti tudi dogodke drugih uporabnikov na njihovem profilu.

  • Svetujem še, da poskrbita, da bo aplikacijo mogoče zagnati tudi v Binderju. Za več podrobnosti glejta Zaključek projekta jan248/Borza#1.

@GhastlyClock
Copy link
Collaborator

Hvala za opombe glede projektne. Nekatere od njih sva uspešno rešila, tako je sedaj možno projekt zagnati tudi z Binderjem.

Bi pa samo omenil, da vsa gesla, ki so vidna v tabeli niso dejanska gesla teh uporabnikov. Namreč ta gesla so bila vnešena skupaj z ostalimi avtomatsko generiranimi podatki in zato so te uporabniki dejansko nedostopni s prijavo. Če pa kdorkoli ustvari nov profil in nastavi geslo, pa se ta ustrezno skrije.

Vsi popravki na projektni so s tem zaključeni in bi vas samo prosila, če lahko posodobite povezavo.

jaanos added a commit to jaanos/OPB that referenced this issue Jul 16, 2020
@jaanos
Copy link
Author

jaanos commented Jul 16, 2020

V redu, sem posodobil.

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