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

Add TTD org in register test data #133

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

martinothamar
Copy link
Contributor

@martinothamar martinothamar commented Dec 20, 2024

Description

Noting this down for now.... For some auth scenarios I've been testing and verifying it's been helpful to be able to lookup ttd from register. Then we can test out org tokens, service owner tokens etc... I think ttd is the only service owner that doesn't have an org nr.

In the case of process/next with a service owner token (ttd being the service owner), with Altinn/app-frontend-react#2889 and Altinn/app-lib-dotnet#880 we can instantiate and process/next with org token for ttd with this change

Related Issue(s)

  • N/A

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)
  • All tests run green

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@ivarne
Copy link
Member

ivarne commented Feb 14, 2025

Is this because of the empty "orgnr" property on https://altinncdn.no/orgs/altinn-orgs.json?

https://github.com/Altinn/altinn-cdn/blob/f080163582c48f5bb62b9ad5d1a6b18659d2c295/orgs/altinn-orgs.json#L745-L762

Should it rather be fixed by adding an orgnr there? All the other service owners are just added based on the json list of orgs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants