Teknisk spesifikasjon for boligselskap. #13
Conversation
heimmste
requested review from
jankfu,
thomasaasen,
jakenilsen,
mikkel-naess and
amund-hh-skatteetaten
docs/api/boligselskap.md
Outdated
| https://{env}/api/boligselskap/v1/{inntektsår}/innsending | ||
| ``` | ||
|
|
||
| ### Eksempel på respons |
There was a problem hiding this comment.
Eksempel på innsending lagt til.
heimmste
force-pushed
the
boligselskap_om_tjenesten
branch
from
fb8cf9d
to
1f9aca4
Compare
|
|
||
| idempotencyKey parameteren er påkrevet. Innholdet skal være en unik UUID. Hvert nye kall til API-et skal ha en | ||
| tilsvarende ny idempotencyKey. Flere etterfølgende POST kall med samme request-body og samme idempotencyKey vil gi den | ||
| samme repsponsen. Kun det første av denne rekken med like API kall vil behandles. IdempotencyKey muliggjør at man trygt |
|
|
||
| ``` | ||
| { | ||
| "leveranse": [ |
There was a problem hiding this comment.
Mulig jeg ser feil her, men er ikke dette en boligsameie melding? Finner blant annet ikke boligselskapsformål..?
There was a problem hiding this comment.
Det er en boligselskapsmelding. Den er til og med hentet fra ekom feeden :)
Hver oppgave har feks boenhet. Boenhet finnes ikke i boligsameie. Boligsameie har matrikkelnummer som ikke finnes i boligselskap.
Men du har rett i at det er felter som ikke er der. Det er jo uheldig i et eksempel kanskje. Kan legge til de som mangler.
There was a problem hiding this comment.
Ok, men som du sier, i så fall så mangler den vel noen felter. Er ganske sikker på at boligselskapsformål må på enten oppgave, leveranse eller begge. Hør gjerne med Jacob ang. dette, han har en oppgave på seg som går på å finne ut av hvordan det skal være. Er litt usikker på hvor langt han har kommet
|
|
||
| ### JSON | ||
|
|
||
| #### Eksempel på innsending |
There was a problem hiding this comment.
En del andre api-eksempler har tøyset til fnr. Det burde kanskje vi også gjøre?
Trenger vi så mange oppgaver i eksemplet??
There was a problem hiding this comment.
Enig at vi bare burde ha med ett par oppgaver i eksempelet
| Følgende scope skal benyttes ved autentisering i Maskinporten: `skatteetaten:boligselskap` | ||
|
|
||
| ## Delegering | ||
| Tilgang til dette API-et kan delegeres i Altinn, f.eks. dersom leverandør benyttes for den tekniske oppkoblingen. Søk opp følgende tjeneste i Altinn for å delegere tilgangen: `Boligselskap API - På vegne av` | ||
|
|
There was a problem hiding this comment.
Er dette riktig?
Dekkes dette av https://skatteetaten.github.io/api-dokumentasjon/om/delegering ?
Åpent spm
There was a problem hiding this comment.
Godt spørsmål. Dette er standardtekst fra de andre APIene. Men det er vel bedre å fjerne det hvis vi ikke vet om det faktisk stemmer.
There was a problem hiding this comment.
Hvem vet dette mon tro?
There was a problem hiding this comment.
Usikker. Men kanskje fjerne den "søk opp tjeneste"-delen da jeg tviler på at det er mulig
Så får vi finne ut av dette etterhvert
There was a problem hiding this comment.
Som prinsipp bør alle apier gjøres delegerbare slik at de som bruker systemleverandør kan få gjort dette. Det er nok også aktuelle selv der det er mest SBSer.
There was a problem hiding this comment.
Ja, dette henviser vel bare til delegering av utsteding av maskingporten-tokens, så i så måte er det jo delegerbart på samme måte som alt annet. Men akkurat nå kan man vel ikke "Søk opp følgende tjeneste i Altinn for å delegere tilgangen: Boligselskap API - På vegne av" - bedre å ta inn teksten når den faktisk stemmer.
heimmste
force-pushed
the
boligselskap_om_tjenesten
branch
from
00b2389
to
1661cf1
Compare
heimmste
force-pushed
the
boligselskap_om_tjenesten
branch
from
1661cf1
to
f106e2b
Compare
Teknisk spesifikasjon for boligselskap. Helt lik den for boligsameie.
Mistet tolmodigheten og la til test- og eksempelsiden også.