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

Overgangsløsning A2: BrokerBridge.GetFileDataByReference #252

Closed
4 tasks done
Tracked by #211
RagnarFatland-Avanade opened this issue Jan 5, 2024 · 6 comments
Closed
4 tasks done
Tracked by #211
Assignees
Labels
kind/user-story Used for issues that describes functionality for our users. product/formidling Issues related til Altinn Broker

Comments

@RagnarFatland-Avanade
Copy link
Collaborator

RagnarFatland-Avanade commented Jan 5, 2024

Implementer følgende metoder i BrokerBridge, med mapping mellom Altinn 2 datamodell og Altinn 3 datamodell.
Anbefaler at denne er første oppgave etter å ha etablert BrokerBridge, da man kan gjøre første mapping mellom A3-representasjonen av en BrokerFile og A2.

  • Metode: GetFileDataByReference
    Aksjoner:
    • Kall GET broker/api/legacy/v1/file/{fileId}
    • Map om til Altinn 2 BrokerServiceFileBE.

Siden dette blir første kall på tvers må det etableres en Maskinportenklient i BrokerBridge, man kan basere seg på eksisterende IMaskinportenHttpClientWrapper.

Dersom det viser seg at nødvendige felter ikke er tilgjengelig i angitt A3 kall, kan isteden man se om det er nødvendig å bruke GET broker/api/legacy/v1/file/{fileId}/details
Dersom det også skulle vise seg å være ufullstendig, tar vi en diskusjon om hvordan vi løser det på best måte og evt. legger till de aktuelle feltene i A3.

  • Vurder å legge til felter i A3-modell
@RagnarFatland-Avanade RagnarFatland-Avanade added kind/user-story Used for issues that describes functionality for our users. product/formidling Issues related til Altinn Broker labels Jan 5, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade added this to the MVP milestone Jan 5, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade added the status/draft Status: When you create an issue before you have enough info to properly describe the issue. label Jan 5, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade moved this from 🆕 New to 📋 Backlog in Altinn melding og formidling Jan 5, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade removed the status/draft Status: When you create an issue before you have enough info to properly describe the issue. label Jan 5, 2024
@hansdragnes hansdragnes moved this from 📋 Backlog to Ready for dev in Altinn melding og formidling Jan 5, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade self-assigned this Jan 8, 2024
@hansdragnes hansdragnes removed this from the MVP milestone Jan 12, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade moved this from Ready for dev to 🏗 In progress in Altinn melding og formidling Jan 15, 2024
@RagnarFatland-Avanade
Copy link
Collaborator Author

RagnarFatland-Avanade commented Jan 16, 2024

Har kommet i gang i A2-kodebasen og har en røff førsteversjon delvis ferdig, men ikke PR-verdig i dag.
Jeg identifiserte noen felter som kan vurderes å legges inn i A3-modellen av ikke bare rent tilbake-kompatible årsaker:

  • File.FileSize - Filstørrelse i bytes
  • File.SentDate / CreatedDate
    • Tidspunkt filen ble Initialisert / opprettet
    • Eksisterer i modellen fra før, bare ikke eksponert i FileOverview.

@RagnarFatland-Avanade
Copy link
Collaborator Author

RagnarFatland-Avanade commented Jan 19, 2024

Felter lagt til i PR #283 som er blitt merget inn.
Jobber videre med resten

@RagnarFatland-Avanade
Copy link
Collaborator Author

Har bedt @tomshag følge opp denne inntil jeg er tilbake på jobb.
Altinn 2 delen krever bare litt cleanup og testing.
Altinn 3 delen får auteniseringsfeil pga ugyldig test-token; må generere ett korrekt ett, samt at tilsynelatende får en uhåndtert feil uten at jeg ser hvordan. Har en unit test som fremprovoserer feilen.

@tomshag
Copy link
Contributor

tomshag commented Jan 22, 2024

@RagnarFatland-Avanade
Da skal Altinn 2 delen være klar, og en pull request er blitt opprettet:
https://dev.azure.com/digdir/Altinn/_git/Altinn/pullrequest/6640

Gjorde følgende endringer:
La inn altinn.config test verdier
La inn ny External type for Status Enum for Recipient, denne viste tidligere til et objekt som så refererte til seg selv. I resultatet i Altinn 3 så returneres kun en Enum der.
Endret på mapper kode, la inn mapping på enkelte verdier som manglet og som trengtes for full utførsel av GetFileOverview.
La inn unittester for GetFileDataByReference
(1 som returnerer fildata og en som ikke returnerer data.)

@tomshag
Copy link
Contributor

tomshag commented Jan 24, 2024

Pull request ferdig.

@tomshag tomshag closed this as completed Jan 24, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in Altinn melding og formidling Jan 24, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade moved this from ✅ Done to 🏗 In progress in Altinn melding og formidling Jan 25, 2024
@RagnarFatland-Avanade RagnarFatland-Avanade moved this from 🏗 In progress to 👀 In review in Altinn melding og formidling Jan 26, 2024
@RagnarFatland-Avanade
Copy link
Collaborator Author

Ferdigstilt endringer for A3 i PR #288
I A2 i PR: https://dev.azure.com/digdir/Altinn/_git/Altinn/pullrequest/6661

@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Altinn melding og formidling Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/user-story Used for issues that describes functionality for our users. product/formidling Issues related til Altinn Broker
Projects
Status: ✅ Done
Development

No branches or pull requests

3 participants