Skip to content

OS2forms SBSYS integration og MitID børne elementerne #286

Closed

Description

@ds-bellcom

Når man bruger elementet "Attachment SBSYS xml", så kan man under "nemid_cpr value" vælge hvilket element der indeholder det CPR nummer, som der skal journaliseres på i SBSYS. Man kan vælge mellem alle elementer i formen af typen "Tekstfelt", men også MitID-elementerne.

Dog kan man ikke vælge nogle af MitID børne elementerne, så hvis man f.eks. gerne vil have en formular hvor en forældre logger ind og afgiver nogle oplysninger der skal journaliseres på et barns CPR nummer, så kan MitID Barn CPR elementet ikke benyttes til dette formål.

Det er specifik disse elementer der ikke kan vælges i XML-elementet:

  • MitID Børn (radioknapper) - #type: os2forms_nemid_children_radios
  • MitID Børn (vælg) - #type: os2forms_nemid_children_select
  • MitID Barn CPR - #type: os2forms_mitid_child_cpr
  • MitID Barn Navn - #type: os2forms_mitid_child_name
  • MitID Barn Adresse - #type: os2forms_mitid_child_address
  • MitID Barn Gade - #type: os2forms_mitid_child_street
  • MitID Barn Husnr. - #type: os2forms_mitid_child_house_nr
  • MitID Barn Etage - #type: os2forms_mitid_child_floor
  • MitID Barn Lejlighedsnr. - #type: os2forms_mitid_child_apartment_nr
  • MitID Barn Postnr. - #type: os2forms_mitid_child_postal_code
  • MitID Barn By - #type: os2forms_mitid_child_city
  • MitID Barn C/O Adresse - #type: os2forms_mitid_child_coaddress
  • MitID Barn Kommunekode - #type: os2forms_mitid_child_kommunekode

Da det er længe siden vi har været inde og kigge på både børne elementerne og "Attachment SBSYS xml" elementet, så har vi brug for 1-2 timer for at kunne estimere opgaven. Er det ok?

@OS2Forms/koordinationsgruppe

Activity

  1. added theissue type on Jan 28, 2026
  2. krpebalk commented on Sep 21, 2026

    @krpebalk

    @ds-bellcom Ballerup betaler estimeringen fra klippekortet.

  3. self-assigned this
    on Sep 21, 2026
  4. pinned this issue on Sep 21, 2026
  5. ds-bellcom commented on Sep 24, 2026

    @ds-bellcom
    ContributorAuthor

    @krpebalk Stan har estimeret på opgaven og det vil tage ca. 11 timer at fikse dette. Vil du vende det med @OS2Forms/koordinationsgruppe og vende retur om det er noget vi skal have sat gang i?

  6. krpebalk commented on Sep 25, 2026

    @krpebalk

    @krpebalk Stan har estimeret på opgaven og det vil tage ca. 11 timer at fikse dette. Vil du vende det med @OS2Forms/koordinationsgruppe og vende retur om det er noget vi skal have sat gang i?

    @ds-bellcom inkl. codereview til ITK?

  7. krpebalk commented on Sep 25, 2026

    @krpebalk

    @gBarlach Kan denne være kandidat til efterårsreleasen?

  8. ds-bellcom commented on Sep 25, 2026

    @ds-bellcom
    ContributorAuthor

    @krpebalk Stan har estimeret på opgaven og det vil tage ca. 11 timer at fikse dette. Vil du vende det med @OS2Forms/koordinationsgruppe og vende retur om det er noget vi skal have sat gang i?

    @ds-bellcom inkl. codereview til ITK?

    @krpebalk sorry, det glemte jeg at skrive noget om. Det er uden review til ITK, men en god tommelfinger regel er at ca. ligger på 10-15% af udviklingstiden, så jeg vil tro at det vil ligge på ca. 1-2 timer fra ITK. Jeg tænker dog ikke at de vil give et estimat endnu, da de ikke kan se hvor meget kode der bliver ændret og heller ikke kan se kompleksiteten i koderettelsen, hvilket er det der afgør hvor meget tid man skal bruge på et review.

  9. ds-bellcom commented on Sep 25, 2026

    @ds-bellcom
    ContributorAuthor

    @gBarlach Kan denne være kandidat til efterårsreleasen?

    Ja, det kunne den sagtens. Jeg markerer den lige med den rette label og som et subissue, så kan den altid flyttes ud igen, hvis den ikke skal med i efterårsreleasen.

  10. krpebalk commented on Sep 25, 2026

    @krpebalk

    @ds-bellcom Den skal ud af efterårsreleasen igen. Ballerup betaler for udviklingen af denne opgave ud fra estimatet på 11 timer + codereview. Vil du sætte det i gang og komme med en tidsplan for hvornår det er er færdigt? Opgaven er på faktura.

  11. ds-bellcom commented on Sep 25, 2026

    @ds-bellcom
    ContributorAuthor

    @krpebalk Super. Jeg beder Stan starte på det med det samme, så jeg håber at vi kan have et PR klar til ITK i starten af næste uge. Hvornår ITK har tid til at lave review kan jeg af gode grunde ikke svare på, men når de har godkendt vores PR, kan vi i starten lægge det på jeres miljøer, som jeg har forstået det - og så vil resten af OS2forms anvenderne få det med ud sammen med efterårsreleasen. Derfor lader jeg den lige stå som en del af efterårsreleasen lidt endnu.

  12. krpebalk commented on Sep 25, 2026

    @krpebalk

    @krpebalk Super. Jeg beder Stan starte på det med det samme, så jeg håber at vi kan have et PR klar til ITK i starten af næste uge. Hvornår ITK har tid til at lave review kan jeg af gode grunde ikke svare på, men når de har godkendt vores PR, kan vi i starten lægge det på jeres miljøer, som jeg har forstået det - og så vil resten af OS2forms anvenderne få det med ud sammen med efterårsreleasen. Derfor lader jeg den lige stå som en del af efterårsreleasen lidt endnu.

    @ds-bellcom Tjek, så kan jeg nå at teste det inden det ryger i releasen :)

  13. unpinned this issue on Sep 28, 2026
  14. ds-bellcom commented on Sep 30, 2026

    @ds-bellcom
    ContributorAuthor

    @krpebalk Stan har været i gang og har lagt rettelsen ind på test.os2forms.dk nu. I kan teste f.eks. på https://test.os2forms.dk/da/form/bellcom-cpr-boern-test

    Brug f.eks. 1012628000 som forældrens CPR nummer og vælg så et af børnene, hvis du vil teste at værdierne kommer rigtigt ned i XML-filen.

  15. krpebalk commented on Oct 1, 2026

    @krpebalk

    @ds-bellcom Det er virkelig fantastisk :) Gud hvor bliver det dejligt at kunne tage dem i brug. Hvornår bliver det lagt på vores testmiljø?

  16. linked a pull request that will close this issueOS-320 SBSYS child elements integration #365on Oct 1, 2026
  17. ds-bellcom commented on Oct 1, 2026

    @ds-bellcom
    ContributorAuthor

    @krpebalk Det ligger også på jeres testmiljø nu :)

  18. ds-bellcom commented on Oct 1, 2026

    @ds-bellcom
    ContributorAuthor

    @gBarlach @MadNicGit vil I lave review på dette PR: #365

  19. jekuaitk commented on Oct 2, 2026

    @jekuaitk
    Contributor

    @ds-bellcom Det ser godt ud!

  20. gBarlach commented on Oct 5, 2026

    @gBarlach

    @MadNicGit ja I må gerne lave review af denne (est. 1 time.)(faktureres på klippekortet)

  21. ds-bellcom commented on Oct 5, 2026

    @ds-bellcom
    ContributorAuthor

    @gBarlach Der er lavet review på denne (af @jekuaitk), vi skal bare have den merged ind nu. Så bolden ligger ved Bellcom :)

  22. gBarlach commented on Oct 21, 2026

    @gBarlach

    Hermed godkendt til efterårsrelease 2, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions