Repository navigation
feat(t20): implement laboratory persistence for LAB-02 and LAB-06 - #56
dbracika29-hash wants to merge 6 commits into
Conversation
|
Pregled 04.11: LAB-02/LAB-06 jesu celine tima 20, ali PR dodaje persistence/DI u zajednički Platform.Authoring modul koji već koristi tim 11. Potrebno je dokumentovati dogovoreni integracioni ugovor sa timom 11 i usklađenost sa LAB-01 identitetom tima 14 te postojećim LAB-02/LAB-06 modulima, bez paralelnih kontradiktornih modela. Sada postoji i merge konflikt sa main: rešiti konflikt, ponovo pokrenuti CI i dostaviti dokaz koordinacije. Ne merge-ujemo u ovom stanju. |
|
Ponovni pregled 04.11. u 13h: LAB-02 i LAB-06 pripadaju timu 20. U main je sada merge-ovan #43 (LAB-02 metadata), a #56 uvodi zaseban persistence sloj u Platform.Authoring koji koristi tim 11. Trenutno postoji merge konflikt. Molimo integrisati sa #43 i LAB-01 modelom tima 14, dokumentovati granicu/ugovor sa timom 11, proveriti izbegavanje duplih modela, rešiti konflikte i ponoviti CI. Do tada ne merge-ujemo. |
|
Ponovni pregled 04.11. uveče: LAB-02/LAB-06 su tim 20, ali #56 i dalje ima merge konflikt i uvodi persistence u zajednički Platform.Authoring modul tima 11. Molimo dokumentovati koordinaciju sa timom 11 i uskladiti sa već merge-ovanim LAB-01/LAB-02 modulima (uključujući današnje #67/#71), bez duplih modela ili DI konflikata. Potreban zeleni CI nakon rešavanja konflikta. |
|
Updated PR #56 after merging the latest main.
Both GitHub checks are green. All 9 PostgreSQL integration tests passed locally. The changes are documented in |
Component and requirement IDs
Changes
AuthoringDbContextand EF Core migrations.ILaboratoryRepositoryfor saving, loading and updating laboratories.Verification
Architecture and data flow
API → Repository → EF Core → PostgreSQL
Security checks and negative tests
Dependencies
Business validation, authorization and cycle detection are not included in this part.