Zaključek projekta - Githubissues

tinmarkon commented 1 year ago

Pozdravljeni, projekt sem tudi jaz zaključil. Imam problem, ker mi ga binder ne pokaže, oziroma vrne server error 500. Nikjer pa ne najdem log datoteke, da bi videl kaj gre narobe.

Če je možno bi imel zagovor ta teden.

jaanos commented 1 year ago

Razlog, da aplikacija ne deluje na Binderju je v knjižnicah pillow in qrcode - da se ti namestita, bo treba v datoteko binder/requirements.txt dodati vrstici pillow in qrcode.

Sicer pa imam še sledeče pripombe:

Slike in CSS na Binderju ne delujejo. Da bodo delovali, bo potrebno v predlogah URL-je (tako kot sicer) podati s funkcijo url - ker gre za statične datoteke, se bo torej treba sklicevati na funkcijo static. Npr. namesto https://github.com/BlackPhoenixSlo/projektna-naloga-Covid-19/blob/4277b38946e4d1eba2119c9366f75dcf8c9d8727/views/pct_certificate.html#L3 bo potrebno narediti
```
<img src="{{url('static', filename='user_qrcodes/user_{}.png'.format(id))}}" class="card-img-top" alt="...">
```
Kolikor vidim, je enaka težava tudi pri povezavi na ikoni levo zgoraj na strani uporabnika (preveri, če je morda še kje).
Vidim, da se v predlogah uporablja rebase (kar je ustrezno), a nekatere predloge poleg tega vsebujejo značko <html> in še enkrat glavo (<head>) s stili. Svetujem, da se stile premakne v CSS, ki se jih uvozi v glavnem dokumentu, potem se pa v posameznih predlogah pojavi samo telo dokumenta, brez glave.
Na stolpcu id_osebe v tabeli uporabnik bi bilo smiselno imeti določilo UNIQUE, da ne bi bilo mogoče imeti več računov za isto osebo.
Povezava O nas v zgornji vrstici uporabnikove strani ne deluje. Če ta stran ni pripravljena, bi bilo bolje to povezavo odstraniti.

Sporoči, ko boš imel pripravljeno končno različico projekta, da posodobim povezavo na repozitoriju predmeta. Za ustni izpit se pa lahko domeniva za petek ob 10h preko Zooma. Sporoči, če ti ustreza, pa pošljem povezavo.

tinmarkon commented 1 year ago

Vse opombe sem popravil, tako da zdaj tudi binder dela. V definicjo tabel, torej schema.sql sem dodal omejitev UNIQUE, ki pa jih zaradi testnih tabel Jake, do katerih nimam pravic ne morem dodati. Jutri bom pognal novo definicijo samo za to tabelo.

Kar se tiče cssjev, sem jih uspešno prestavil in odstranil vse glave. Edino težavo, ki jo še imam je v datoteki my_pacients.html. Nekako mi stvari na vrhu (tekst + del slik) odreže in jih ne prikazuje pravilno. Je tukaj problem v classu diva, ki ga imam na vrhu?

jaanos commented 1 year ago

Izgleda, da je težava v pravilih iz v signin.css. Svetujem, da jih vključiš samo tam, kjer jih potrebuješ - torej premakneš ustrezno značko <link> iz base.html v login.html.

Mimogrede, vidim, da bo treba še tukaj podati pot z url: https://github.com/BlackPhoenixSlo/projektna-naloga-Covid-19/blob/a0458c27589793255c8955205a76d5eb6cd1c4ec/views/base.html#L7

tinmarkon commented 1 year ago

Projekt sem popravil in je zaključen. Če je možno bi potrdil razgovor v petek.

jaanos commented 1 year ago

Sem posodobil povezavo na repozitoriju predmeta. Opazil sem sicer, da dodajanje pacienta ne deluje - pri podatkih iz README.md aplikacija javi, da se ne ujemajo (nisem preverjal v bazi, če je res tako); ko sem pa odstranil pacienta in ga znova dodal, pa sem dobil napako.

Povezavo za petek bom poslal po e-pošti.

tinmarkon commented 1 year ago

Posodobil sem kodo in zdaj bi moralo delovati tudi to. Poleg tega sem dodal še osebo, ki se lahko doda v bolnišnico. Če je možno bi prosil za posodobitev povezave.

jaanos commented 1 year ago

V redu, sem posodobil povezavo.

@tinmarkon, sem ti poslal povezavo za Zoom na oba e-poštna naslova (@student.fmf.uni-lj.si in @student.uni-lj.si) in sem dobil obvestilo, da ni moglo dostaviti na FMF-jev naslov. Če nisi prejel sporočila, me prosim kontaktiraj po e-pošti.

BlackPhoenixSlo / projektna-naloga-Covid-19

Zaključek projekta #5