search

bitsnorge / e-tinglysing-afpant

AFPANT - Altinn Formidlingstjenester for kjøpers e-signerte pantedokument
4 stars 13 forks source link

added new message type GjennomfoertEtinglysing to xsd and xslt #36

Closed koeide-visma closed 3 years ago

koeide-visma commented 3 years ago

Testmelding ser slik ut etter transform med xslt endringene: image

koeide-visma commented 3 years ago

Pull request ser bare ut til å innholde xsd oh xslt, burde vel også følge en liten oppdatering i selve dokumentasjonen. Tenker tilsvarende dette: https://github.com/bitsnorge/e-tinglysing-afpant/blob/master/spesifikasjoner/afpant/afpant-intensjon/afpant-intensjon.md

Har ingen innvendinger på xsd og xslt så setter godkjent

Veldig bra Jon Egil :) Jeg er enig, vi bør oppdatere dokumentasjon også, kan evt se på dette neste uke.

koeide-visma commented 3 years ago

hvis ingen har innvendinger her innen i ettermidag, så kjører jeg en merge til master rundt kl 15 :) Så kom med feedback før den tid om noen ønsker å justere noe.

krisho98 commented 3 years ago

Fint om dokumentasjonen også oppdateres før vi merger tenker jeg. (slik at det ikke blir glemt)

koeide-visma commented 3 years ago

@krisho98 det er ok, jeg avventer merge. Skal prøve å få til et utkast til dokumentasjon neste uke :)

javaguruen commented 3 years ago

Synes noe er uklart fra diskusjon på issues og her:

  1. Skal kartverket sin XML ligge som en egen fil i zip-arkivet eller som et base64-encoded element i XML-en som er meldingen?
  2. Hvis den skal ligge som selvstendig fil i arkivet, trenger den ikke base-64 encodes.
  3. Filen(e) som skal ligge i zip-arkivet må ha en regel for hvordan de skal navngis.
  4. Denne senders reference det er snakk om å mappe i matrise til dokumentreferansen i kartverket sitt svar, forstår det slik at det er senders reference som bank brukte da sendte pantedokumentet til megler. Vi bruker bankens referanse inne i SDO-en (pantedokumentet) som dokumentreferanse ved tinglysing og som også i svaret tilbake fra Kartverket når det er tinglyst. Så egentlig burde det være nok informasjon for banken å koble tinglysingsinformasjonen på uten å ha en mapping til senders reference.

Dette er fint om vi tydeliggjør i dokumentasjonen som skal skrives.

koeide-visma commented 3 years ago

Da er det lagt til .md dokumentasjon for gjennomfoertetinglysing :) @javaguruen jeg tror dokumentasjon nå skal dekke de punktene du nevnte over, vi tar med xml fra Kartverket som et vedlegg og da som application/xml og ikke base64 encodet. Har også tatt med noen krav til filnavn og prøvd å tydeliggjøre at sendersReference er fra SignedMortgageDeed meldingen og at vi bruker samme dokumentreferanse som banken sender i tinglysing til KV.

@JonEgilH @ropatambitadotcom @paal-havnevik, @krisho98 ser dere også over .md filen at det ser ok ut?

javaguruen commented 3 years ago

Ser at det er kommet krav fil filnavn. Det er bra, men burde vi ikke prøvd å ha like konvensjoner/krav til filnavn på alle meldingstypene? Hvis du ser på kravene til pantedokumentet (https://github.com/bitsnorge/e-tinglysing-afpant/blob/master/spesifikasjoner/afpant/afpant-kj%C3%B8perspantedokument/afpant-kj%C3%B8perspantedokument-1-0-0.md) så er de:

Burde ikke filene hete "forsendelsesstatus*.xml" og "gjennomfoertetinglysing.xml hvor kan erstattes med vilkårlig streng slik at det blir mer likt det vi har fra før?

JonEgilH commented 3 years ago

Sendt ut på epost, men legger det inn her så det er public for alle

Kom på en ting som vi kanskje ikke har tenkt på. I noen tilfeller kan jo banken sende flere dokumenter til megler, skjer vel ikke så ofte, men kan skje. Ble vel faktisk nevnt på møtet i går at f.eks HB pleide å gjøre dette. Bankene kan til og med sende flere dokumenter på samme eiendom.. Det er da separate forsendelser til dere og vil ha hver sin «dokumentreferanse» og hver sin «altinnsendersreference».

I GjennomfoertEtinglysing har man lagt inn «dokumentreferanse» og «altinnsendersreference». Disse kan altså bare komme en gang i svaret.

<gjennomfoertetinglysingdetaljer>
        **<dokumentreferanse>521450_291720049057</dokumentreferanse>**
        <altinnsendersreference>8899644559057</altinnsendersreference>
    </gjennomfoertetinglysingdetaljer>

MetaData har en vedleggsliste så den kan i teorien inneholde flere «forsendelsesstatus-xxx.xml» «forsendelsesstatus» er vel bygget opp av svaret dere har fra kartverket og vil da gjelde hele pakken, dvs den vil vel inneholde flere dokumenter og ikke bare det dokumentet som tilhører vår «altinnsendersreference» ettersom jeg egentlig har to av disse i gitte situasjoner.

Mao GjennomfoertEtinglysingDetaljer er ett dokument, Metadata kan ha en liste med dokumenter og hvert av disse kan ha dokumenter fra flere «altinnsendersreference»

Forsendelses status inneholder allerede flere dokumenter (hjemmelsovergang og pantedokumentet) men vil vel da i verste fall også inneholde flere pantedokument.

<forsendelsesstatus xmlns=http://kartverket.no/grunnbok/wsapi/v2/domain/innsending>
<behandlingsutfall>TINGLYST</behandlingsutfall>
    <saksstatus>AVSLUTTET</saksstatus>
    <saksnummer>19019509</saksnummer>
    <tinglysingsinformasjon>
        <dokumentinformasjon>
            <dokumentinformasjon>
                <dokumentreferanse>24afba24-ca07-41eb-bacd-07ff803e27b5</dokumentreferanse>
                <dokumentaar>2021</dokumentaar>
                <dokumentnummer>200161805</dokumentnummer>
                (hjemmelsovergang)
<dokumentinformasjon>
                **<dokumentreferanse>521450_291720049057</dokumentreferanse>**
                <dokumentaar>2021</dokumentaar>
                <dokumentnummer>200161806</dokumentnummer>
                <embetenummer>200</embetenummer>
                <rettsstiftelsesinformasjon>
                    <rettsstiftelsesinformasjon>
                        <rettsstiftelsesreferanse>OB_PDO.510462530_I057X07_55540100</rettsstiftelsesreferanse>

(bankens ene pantedokument i dette tilfellet)

Vi får jo i retur dokumentreferanse 521450_291720049057 per dokument og i dette tilfellet stemmer det med info i gjennomfoertetinglysingdetaljer, men hadde vi to dokumenter ville det ikke ha stemt . Vi kan altså klare å lete oss frem til riktig dokument her, men strukturen på svaret gir da ikke helt mening. Betyr det at vi bør se igjennom GjennomfoertEtinglysing typen?

Er enkleste løsning av gjennomfoertetinglysingdetaljer blir en liste? Evt at man sier at man alltid skal få ett svar per «altinnsendersreference». Det betyr at megler må sende samme «forsendelsesstatus» flere ganger hvis det er flere dokumenter i oppdraget.

koeide-visma commented 3 years ago

@JonEgilH, jeg tror egentlig det skal gå fint med flere pantedokumenter fra samme bank slik vi har implementert løsningen nå :) Vi sender aldri mer enn 1 vedlegg med gjennomfoertetinglysing, siden meldingen vil gjelde for 1 spesifikt pantedokument i 1 spesifikk forsendelse til KV.

Gitt at en bank støtter GjennomfoertEtinglysing i Akeldo, og sender oss pantedok A og pantedok B til samme megleroppdrag. Dersom begge disse dokumentene da tinglyses av megler, og de ender opp med status Tinglyst OK hos KV, så skal banken få 2 gjennomfoertetinglysing meldinger tilbake, dvs 1 for hvert pantedokument (med det aktuelle pantedokumentets dokumentreferanse og Altinn senders ref fra SignedMortgageDeed forsendelsen i hvert tilfelle). Dersom både dok A og B ble tinglyst i samme forsendelse til Kartverket vil hver av gjennomfoertetinglysing meldingene ha samme vedlegg og banken vil få begge meldingene omtrent samtidig (siden det er samme forsendelsesstatus fra KV). Hvis de derimot tinglyses i separate forsendelser med ulike tidspunkt vil banken også få 2 gjennomfoertetinglysing meldinger, men på forskjellige tidspunkt og med forskjellig vedlegg.

Det som forøvrig er viktig er at hvert pantedokument har en unik dokumentreferanse når de er sendt til samme megleroppdrag (ellers feiler løsningen på vår side hvertfall og logger internt at det ikke er støttet, dersom det er flere dokumenter med samme dokumentreferanse knyttet til samme megleroppdrag), men stemmer det at det skulle være dokumentert et sted på github at disse dokumentreferansene må være unik?

Høres det ok ut at vi beholder løsningen for gjennomfoertetinglysing som skissert over? Vi kan gjerne kjøre ende-til-ende test på dette også (2 pantedok fra bank, både i samme forsendelse og separate forsendelser til KV), spesielt hvis det er et relativt vanlig scenario og det er relativt greit for dere å teste, Jon Egil :) Dere kan da f.eks produsere et ekstra pantedokument til det megleroppdraget vi kjørte siste testen på nå, og så tester vi begge casene (samme/separat etgl forsendelse) på dette oppdraget.

koeide-visma commented 3 years ago

Ser at det er kommet krav fil filnavn. Det er bra, men burde vi ikke prøvd å ha like konvensjoner/krav til filnavn på alle meldingstypene? Hvis du ser på kravene til pantedokumentet (https://github.com/bitsnorge/e-tinglysing-afpant/blob/master/spesifikasjoner/afpant/afpant-kj%C3%B8perspantedokument/afpant-kj%C3%B8perspantedokument-1-0-0.md) så er de:

* Eventuelt følgebrev må følge konvensjonen: "coverletter_*.[pdf|xml]". Filtype må samsvare med valgt verdi i 'coverLetter'.

* Pantedokumentet må følge konvensjonen: "signedmortgagedeed__.[sdo|xml]" Wildcard "_" kan erstattes med en vilkårlig streng (må være et gyldig filnavn), f.eks lånesaksnummer eller annen relevant referanse for avsender.

Burde ikke filene hete "forsendelsesstatus.xml" og "gjennomfoertetinglysing.xml hvor * kan erstattes med vilkårlig streng slik at det blir mer likt det vi har fra før?

@javaguruen, jo enig i at navnestandard bør være konsistent, jeg justerer dette i dokumentasjonen og implementasjonen vår slik at det blir iht SignedMortgageDeed.

En liten digresjon for øvrig er jo at det er litt synd at SignedMortgageDeed ikke følger helt samme standard og retningslinjer som resten av meldingene vi har lagt til i ettertid. Den har ingen xsd, pantedokumentet burde kanskje vært et vedlegg heller og meldingstypen burde hatt noen av dataene som pt sendes som manifest properties, etc. Men like greit/ingenting i veien for at den kan være et utganspunkt for navnekonvensjoner likevel slik jeg ser det :)

koeide-visma commented 3 years ago

@javaguruen, @JonEgilH jeg oppdaterte dokumentasjon med samme filnavn konvensjon som SignedMortgageDeed + et ekstra notat for tilfeller med flere pantedokumenter fra samme bank i felles forsendelse til KV. Ser dette ok ut?

JonEgilH commented 3 years ago

@JonEgilH, jeg tror egentlig det skal gå fint med flere pantedokumenter fra samme bank slik vi har implementert løsningen nå :) Vi sender aldri mer enn 1 vedlegg med gjennomfoertetinglysing, siden meldingen vil gjelde for 1 spesifikt pantedokument i 1 spesifikk forsendelse til KV.

@ropatambitadotcom @javaguruen greit om dere også bekrefter denne oppfattelsen. Det er helt supert for meg å få flere forsendelser slik at jeg får en par sendersRef selvom de fra megler har inngått i samme pakke til kartverket.

koeide-visma commented 3 years ago

PS. En liten utfordringen jeg oppdaget når jeg skulle justere navnekonvensjonen i koden vår: Det ser ut som eneste navnekrav vi hadde på filer fra før var på SignedMortgageDeed fra bank til megler (og ikke intensjonsmeldinger eller svar på SignedMortgageDeed fra megler). Og jeg ser at vi sender alle våre filer fra megler til bank på følgende format: Websystemer{ny GUID}{navn på meldingstype pascal case, f.eks IntensjonFraMegler}.xml

Dermed må vi enten justere dette for alle meldingstyper eller bare spesialhåndtere gjennomfoertetinglysing inntil videre. @JonEgilH @ropatambitadotcom @ArildFlatland, ser noen av dere noen som helst issues på deres side med at vi endrer til følgende format? meldingstypelowercaseWebsystemer{ny GUID}.xml

Dersom vi er usikker må vi kanskje spesialhåndtere gjennomfoertetinglysing inntil videre.

JonEgilH commented 3 years ago

PS. En liten utfordringen jeg oppdaget når jeg skulle justere navnekonvensjonen i koden vår: Det ser ut som eneste navnekrav vi hadde på filer fra før var på SignedMortgageDeed fra bank til megler (og ikke intensjonsmeldinger eller svar på SignedMortgageDeed fra megler). Og jeg ser at vi sender alle våre filer fra megler til bank på følgende format: Websystemer{ny GUID}{navn på meldingstype pascal case, f.eks IntensjonssvarFraMegler}.xml

Dermed må vi enten justere dette for alle meldingstyper eller bare spesialhåndtere gjennomfoertetinglysing inntil videre. @JonEgilH @ropatambitadotcom @ArildFlatland, ser noen av dere noen som helst issues på deres side med at vi endrer til følgende format? meldingstypelowercaseWebsystemer{ny GUID}.xml

Dersom vi er usikker må vi kanskje spesialhåndtere gjennomfoertetinglysing inntil videre.

Kan ikke se at det skaper problemer for oss

koeide-visma commented 3 years ago

PS. En liten utfordringen jeg oppdaget når jeg skulle justere navnekonvensjonen i koden vår: Det ser ut som eneste navnekrav vi hadde på filer fra før var på SignedMortgageDeed fra bank til megler (og ikke intensjonsmeldinger eller svar på SignedMortgageDeed fra megler). Og jeg ser at vi sender alle våre filer fra megler til bank på følgende format: Websystemer{ny GUID}{navn på meldingstype pascal case, f.eks IntensjonFraMegler}.xml Dermed må vi enten justere dette for alle meldingstyper eller bare spesialhåndtere gjennomfoertetinglysing inntil videre. @JonEgilH @ropatambitadotcom @ArildFlatland, ser noen av dere noen som helst issues på deres side med at vi endrer til følgende format? meldingstypelowercaseWebsystemer{ny GUID}.xml Dersom vi er usikker må vi kanskje spesialhåndtere gjennomfoertetinglysing inntil videre.

Kan ikke se at det skaper problemer for oss

ok supert :) fint om dere bekrefter at det vil funke for Nordea og Ambita sine banker også @ArildFlatland og @ropatambitadotcom

ArildFlatland commented 3 years ago

Hei,

Kan ikke se at det skal skape noe problem på vår side heller.

Med vennlig hilsen / best regards

Arild Flatland IT Expert Developer

Nordea | Software Development / N-Core Development - E Besøk meg: Essendrops gate 7, 0368 Oslo Skriv til meg: Postboks 1166, Sentrum, 0107 Oslo Mob: +47 917 42 690 E-post: @.**@.> Hjemmeside: nordea.nohttp://www.nordea.no/

Nordea Bank AB (publ), filial i Norge, Essendrops gate 7, Postboks 1166 Sentrum, 0107 Oslo, 983258344 MVA (Foretaksregisteret) Nordea Bank AB (publ), Stockholm, 516406-0120 (Bolagsverket)

Denne e-posten kan inneholde konfidensiell informasjon. Hvis du har mottatt den ved en feiltakelse, vennligst si fra til avsender og slett e-posten. Ikke del innholdet med andre eller kopier det.

From: koeide-visma @.> Sent: Tuesday, March 23, 2021 11:34 AM To: bitsnorge/e-tinglysing-afpant @.> Cc: Flatland, Arild @.>; Mention @.> Subject: Re: [bitsnorge/e-tinglysing-afpant] added new message type GjennomfoertEtinglysing to xsd and xslt (#36)

PS. En liten utfordringen jeg oppdaget når jeg skulle justere navnekonvensjonen i koden vår: Det ser ut som eneste navnekrav vi hadde på filer fra før var på SignedMortgageDeed fra bank til megler (og ikke intensjonsmeldinger eller svar på SignedMortgageDeed fra megler). Og jeg ser at vi sender alle våre filer fra megler til bank på følgende format: Websystemer{ny GUID}{navn på meldingstype pascal case, f.eks IntensjonFraMegler}.xml Dermed må vi enten justere dette for alle meldingstyper eller bare spesialhåndtere gjennomfoertetinglysing inntil videre. @JonEgilHhttps://github.com/JonEgilH @ropatambitadotcomhttps://github.com/ropatambitadotcom @ArildFlatlandhttps://github.com/ArildFlatland, ser noen av dere noen som helst issues på deres side med at vi endrer til følgende format? meldingstypelowercaseWebsystemer{ny GUID}.xml Dersom vi er usikker må vi kanskje spesialhåndtere gjennomfoertetinglysing inntil videre.

Kan ikke se at det skaper problemer for oss

ok supert :) fint om dere bekrefter at det vil funke for Nordea og Ambita sine banker også @ArildFlatlandhttps://github.com/ArildFlatland og @ropatambitadotcomhttps://github.com/ropatambitadotcom

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/bitsnorge/e-tinglysing-afpant/pull/36#issuecomment-804794285, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALIOL2PXBEU2F6O2N77B5TTTFBVB7ANCNFSM4XMSF23Q.

koeide-visma commented 3 years ago

Da merger jeg denne slik vi ble enig om på prosjektmøtet idag :)