[matkezza]

Component and requirement IDs

• Component: REV-05 - domen i baza za dvostepeno odobravanje objave.
• Requirements: REV-05; povezani zahtevi ADM-02 i SEC-08 u okviru ovog dela.
• Author(s): Matija Brkusanin, PR101/2023, Tim 4.

Changes

• Dodati statusi revizije i kontrolisani prelazi, politika kursa i zabrana samoodobravanja kada je nezavisno odobrenje obavezno.
• Dodati SHA-256 ContentHash, RowVersion i transakciono cuvanje odluke zajedno sa stanjem i istorijom.
• Dodati PostgreSQL/EF Core mapiranja i migracije CreateReviewTables i AddRevisionState.
• Domen izdvojen iz API projekta; dodata registracija persistence servisa i CI provera.

Verification

• Lokalno prolazi 61 jedinicni/modelski i 20 PostgreSQL integracionih testova u Release rezimu.
• API Release build prolazi bez upozorenja i gresaka.
• Nema promena modela koje nedostaju u migracijama.
• Proverene su prazna instalacija, ponavljanje migracije i nadogradnja uz ocuvanje podataka.
• GitHub CI treba potvrditi na ovom PR-u.

Architecture and data flow

Modeli, odgovornosti klasa, dijagrami i komande opisani su u docs/components/REV-05.md. Domain nema zavisnosti od HTTP-a i EF Core-a; Infrastructure cuva podatke preko PostgreSQL-a.

Security checks and negative tests

Provereni su zabrana samoodobravanja, nedostajuca politika, promenjen hash, zastarela verzija i nedozvoljeni prelazi. Test dva konkurentna zahteva potvrdjuje samo jednu uspesnu odluku i jedan zapis istorije. Neuspeh upisa istorije ponistava i promenu statusa/verzije. Lozinke i lokalni .env nisu ukljuceni.

Dependencies

Ovaj PR obuhvata domen i bazu. Autentikacija, autorizacija nad konkretnim resursom, HTTP endpointi, REV-01 snapshot/numerisanje, REV-02 validacija i audit objavljivanja ostaju povezivanje sa komponentama njihovih autora. Identifikatore i snapshot ugovor treba usaglasiti prilikom integracije.

[owlCoder]

Pregled: REV-05 je dodeljen timu 4; CI domain/model, PostgreSQL integration i migration checks su uspešni. Nakon merge-a #33, PR ima konflikt sa aktuelnim main. Molimo usklađivanje grane, rešavanje konflikata i ponavljanje CI pre merge-a.

[owlCoder]

Ponovni pregled (03.11): REV-05 pripada timu 4, a kompletan GitHub CI (model, PostgreSQL integracija, migracije) je zelen. Trenutno su prisutni konflikti sa ažuriranim main. Molimo uskladiti granu i ponoviti CI; bez toga merge nije moguć.

[owlCoder]

Ponovni pregled 03.11. uveče: REV-05 je i dalje legitimna celina tima 4 i poslednji CI (uključujući PostgreSQL integracione testove i migracije) je zelen. Posle novih merge-ova u main (#51, #48, #28 i #47), GitHub trenutno prijavljuje merge konflikt. Molimo sinhronizovati sa aktuelnim main, rešiti konflikt bez uklanjanja tuđih modula i ponoviti kompletan CI. PR ostaje otvoren.