vanjadimkovska / ReservationApplication

0 stars 0 forks source link

backend & springboot #3

Open sysmat opened 7 months ago

sysmat commented 7 months ago
/api
  /vx - opcijsko določimo verzijo
      /room
         POST, GET, DELETE  roomId
         GET  /all
     /reservation
      POST, GET, DELETE  reservationId
vanjadimkovska commented 7 months ago

Zadnjo verzijo (3.2.0) nisem uporabila zaradi napake ki se mi je pojavila med prvim zagonom programa, zaradi nekih konfliktov, pa sem se spomnila da sem imela skoraj neki projekt na fakulteti in da sem imela isti problem in sem kr šla in pogledala katero verzijo sem takrat uporabila, in uporabila tisto. javax.persistence sem uporabila zaradi napak z podatkovno bazo in žal nisem vedla da je pokrito z spring-boot-starter-data-jpa. Tako kot sem sem že napisala zgoraj, vsa neujemanja s najnovejšimi verzijami so zaradi napak in konfliktov z nekimi drugimi knjižnicami (slf4j-api). Glede kreiranja sob v mainu, sem to največ naredila da bi najlažje videli kako vse deluje, da ne bi rabili ročno ustvarjati, drugače pa sem preiskala da se to lahko naredi z preverjanjem aktivnega profila v aplikaciji. Za @CrossOrigin žal nisem poznala drugega načina, in sem to naredila da bi se izognila napak z CORS.

sysmat commented 7 months ago
vanjadimkovska commented 7 months ago

Pozdravljeni, Se opravičujem zaradi malo poznega odgovora ampak sem se zaradi obveznosti komaj danes spravila v reševanje. Spremembe so vidne na git-u. Lep pozdrav

kle-pra commented 7 months ago

Glede cors-a se drugače za development server lahko izognete tudi z uporabi proxy-ja, za vaš primer v vite: https://vitejs.dev/config/server-options.html#server-proxy

  1. Z docker compose je sedaj precej bolj elegantno.
  2. Tudi verzije so sedaj urejene, zadnja verzija Spring Boot, slf4j.
    • Iz lib, ali lahko pobrišete jar-e saj to načeloma ne bi smelo biti več potrebno? Pa tudi drugače v maven projektih libov ne dodajamo tako.
  3. Vrhnja anotacija v controller tudi fajn, tudi naredi controller bolj berljiv. Poimenovanja pathov so sedaj tudi bolj po priporočilih za REST.
  4. Fajn, da ste dodali preverjanje profila prek Environment - alternativno je opcija tudi z @Profile anotacijo nad metodo.

Vidim, da ste iz .gitignore odstranili target, kakšen razlog? Drugače to ni priporočjivo, saj ni potrebe, da je ta zbuildana koda v git, saj zaseda prostor (lahko se ustvari in zbuilda po potrebi lokalno ali v sklopu CI pipeline-a), pa tudi commiti so potem težje berljivi, ker je toliko sprememb (target vsebuje kup datotek). Boljše, da se spet doda.

Hvala za dodane popravke, ter tudi zgornje odgovore, ki so v glavnem v redu. Trenutno smo še v fazi izbire, vas pa bomo kmalu obvestili glede naprej.

vanjadimkovska commented 7 months ago

Popravki glede lib, target pa server proxy so vidni na repozitoriju.

Hvala na nasvete in smernice in upam da se kmalu slišimo.

sysmat commented 7 months ago