Appointment

[thomiz]

Har ikke sett igjennom i detalj, noen spørsmål @oaassv :

• Hva betyr det at flere profiler ikke heter no-basis?
• Navngivningen (filer og id) er ikke i tråd med prinsipper no-basis
• Skal utvidelsene også være no-basis?
• Det ser ut som snapshots er lastet opp, men vi ønsker å basere oss på differentials

[oaassv]

@thomiz

• vi endrer navn til no-basis - ekstensjonene tenker vi er no-basis (og knyttet til profil). Jeg ser vi har sluttet å kvalifiere ekstensjon med navn på ressurs (som det for eksempel ble gjort for no-basis-person-citizenship)
• vi legger ut nye filer som er differntials

[thomiz]

Dette går ikke gjennom build prosessen: https://github.com/thomiz/fsh-no-basis/runs/4249565347?check_suite_focus=true

Dere må holde extensions id'er til lowercase, uten camelcase. både i extension.id og filnavn. Da er det kanskje også like greit å holde det lowercase også i profilene som henviser til disse extensions.

Gjelder alle tre extensions, men i det siste bygget fikset jeg på partOf til partof bare for å teste.

[oaassv]

Ligger på egen Appointment branch På fredag 19. november 2021, 08:06:35 CET skrev Espen Stranger Seland @.***> følgende:

@rockphotog commented on this pull request.

Er dette et utkast på noe som ikke bør legges til master?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

[thomiz]

Er dette et utkast på noe som ikke bør legges til master?

Extensions må gjennomgås om de er aktuelle for basisprofiler vs områdeprofil.

@rockphotog Er det andre ting i branchen du tenker vi må gjøre før vi eventuelt pusher innholdet til master?

Når endringene har vært på høring og er godkjent kan vi lage en ny release som erstatter v2.1.1 av no-basis. Det vil da bli versjon 2.2.0.

[thomiz]

Denne pull'en er nå bare rot, med doble filer blant annet.

Kan dere rydde opp og lage en ny pull request, eventuelt basert på en ny branch. @oaassv @lfreddy