Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Komplettere OpenAPI dokumentasjon #1239

Closed
prange opened this issue Oct 7, 2024 · 2 comments
Closed

Komplettere OpenAPI dokumentasjon #1239

prange opened this issue Oct 7, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@prange
Copy link

prange commented Oct 7, 2024

Introduction

OpenAPI spesifikasjonen er ikke fullstendig og vi anmoder om at det innføres rutiner som i større grad sørger for at kode og spec er i synk og tilgengelig fortløpende.

Description

OpenAPI spesifikasjonen og Swaggerdokumentene er uvurderlige kilder til informasjon om integrasjon mot Dialogporten. Spesielt nå som API'et fortsatt endres relativt hyppig er det veldig nyttig for oss som tester ut Dialogporten å ha god og nøyaktig OpenAPI-spesifikasjon slik at vi enkelt kan henge med og evt autogenerere klient-kode.
F.eks. er returtypen til GET /api/v1/serviceowner/dialogs/{dialogId}/transmissions bare en string i spesifikasjonen.

I tillegg er typespesifikasjonene unike for hvert endepunkt slik at samme datastruktur er navngitt ulikt for for f.eks. GET /dialogs og GET /dialogs/{dialogId}.

En alternativ løsning er her jo å publisere datatypene som en NuGet pakke fortløpende (#1137).

Jeg vil tro å få på plass dette kan spare dere for en del spørsmål om APIet framover.

@oskogstad
Copy link
Collaborator

Vi har en form for automatisk sjekk for dette (snapshot test av OpenAPI-dokumentet), men ingen sjekk for å dekke akkurat denne feilen du fant med transmissions ... skal se på det. 🤔
Vet og vi har en del feil/mangler rundt hvilke statuskoder som kan komme fra hvert endepunkt.

GET /dialogs og GET /dialogs/{dialogId} har ikke samme datastruktur, forsto ikke helt hva du mente her?
(Search-endepunktet (GET /dialogs) har en hel del properties fjernet, f.eks. alt av attachments, transmissions, gui-actions)

#1137 skal vi muligens se på i neste sprint som starter straks 😊

Har fiksa transmissions sin returtype, skal få pusha asap 🚀
Takk for feedback! 💪🏼

oskogstad added a commit that referenced this issue Oct 10, 2024
…nAPI spec (#1244)

<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->

## Related Issue(s)

- #1239 

## Verification

- [ ] **Your** code builds clean without any errors or warnings
- [ ] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and
we'll help out)

## Documentation

- [ ] Documentation is updated (either in `docs`-directory, Altinnpedia
or a separate linked PR in
[altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if
applicable)


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Introduced multiple new schemas for dialog activity and transmission
handling, enhancing metadata management.
- Updated API endpoints to return detailed lists of dialog activities
and their attachments.
- Enhanced data structures with new properties for better clarity and
functionality in dialog activities and transmissions.
- Improved Swagger configurations for better documentation clarity and
specificity for dialog activity and transmission endpoints.
- Added support for notification conditions in the API, improving the
handling of related queries.

- **Bug Fixes**
- Improved response structure of dialog activity and transmission
endpoints to ensure accurate data retrieval.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@prange
Copy link
Author

prange commented Oct 11, 2024

Ref datatyper: Det er enkelte datatyper som er strukturelt identiske, men har ulike navn avhengig av endepunkt og plassering, f.eks ***ActorDto, ***AttachmentDto eller ***ActivityDto (ta høyde for at jeg ha blingsa og ikke sett de strukturelle forskjellene)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

3 participants