Skip to content

feat: Add parsing definitions from the api calls#1252

Merged
SjurSutterudSagen merged 12 commits into
mainfrom
feature/PxWeb2-1037-add-collecting-definitions-from-the-api-calls
May 11, 2026
Merged

feat: Add parsing definitions from the api calls#1252
SjurSutterudSagen merged 12 commits into
mainfrom
feature/PxWeb2-1037-add-collecting-definitions-from-the-api-calls

Conversation

@SjurSutterudSagen
Copy link
Copy Markdown
Contributor

@SjurSutterudSagen SjurSutterudSagen commented Apr 30, 2026

Add parsing for the definitions in the metadata API call into the jsonStat2ResponseMapper, make the TableInformation component use that for the DefinitionsTab.

This will make the DefinitionsTab show up again, if the API has the correct definitions in the response.

SjurSutterudSagen and others added 6 commits April 29, 2026 13:10
Still need to figure out the state handling, since when i console log what is returned it returns correctly, and then runs again (for the data?) and empties the state with the definitions.

Co-authored-by: Copilot <copilot@github.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

Branch Deployment to Cloudflare Pages

🚛 Preview URL: https://b1296e3f.pxweb2.pages.dev
🚧 Branch preview URL: https://feature-pxweb2-1037-add-coll.pxweb2.pages.dev
🚚 Alternative Preview URL: https://a1ff03bb.pxweb2.pages.dev
👷 Alternative Branch preview URL: https://alternative-feature-pxweb2-1-qi0t.pxweb2.pages.dev

@SjurSutterudSagen SjurSutterudSagen changed the title Feature/px web2 1037 add collecting definitions from the api calls feat: Add parsing definitions from the api calls May 8, 2026
@SjurSutterudSagen SjurSutterudSagen marked this pull request as ready for review May 8, 2026 09:40
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@MikaelNordberg MikaelNordberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@SjurSutterudSagen SjurSutterudSagen merged commit 31f7060 into main May 11, 2026
14 checks passed
@SjurSutterudSagen SjurSutterudSagen deleted the feature/PxWeb2-1037-add-collecting-definitions-from-the-api-calls branch May 11, 2026 13:11
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