Skip to content

Feature/new metaid in json2 serializer#197

Merged
JohannesFinsveen merged 8 commits intomainfrom
feature/new-metaid-in-json2-serializer
Apr 24, 2026
Merged

Feature/new metaid in json2 serializer#197
JohannesFinsveen merged 8 commits intomainfrom
feature/new-metaid-in-json2-serializer

Conversation

@JohannesFinsveen
Copy link
Copy Markdown
Collaborator

@JohannesFinsveen JohannesFinsveen commented Apr 15, 2026

Bumbs PxWeb.Api2.Server.Models to version "2.3.2" as a sideeffect

In the same way as the metadata endpoint, the data endpoint now outputs links generated from metaid.

Support for the OldWay of just outputting the urns, will hopfully be dropped in 3.0, so the NewWay does not reuse things like collection of urns from the OldWay.

@JohannesFinsveen JohannesFinsveen marked this pull request as ready for review April 23, 2026 08:06
likp
likp previously approved these changes Apr 24, 2026
Copy link
Copy Markdown
Member

@runejo runejo left a comment

Choose a reason for hiding this comment

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

@JohannesFinsveen
Consider https://www.conventionalcommits.org/en/v1.0.0/ when creating PR title

Compare the files in this serializer with the corresponding files in PxWebApi and try to minimize the difference

PCAxis.Serializers/JsonStat2/Model/JsonStat2Dataset.cs
PxWebApi/PxWeb/Models/Api2/DatasetSubclass.cs
PCAxis.Serializers/JsonStat2Serializer.cs
PxWebApi/PxWeb/Mappers/DatasetMapper.cs

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@runejo runejo left a comment

Choose a reason for hiding this comment

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

💯

@JohannesFinsveen JohannesFinsveen merged commit 8fe16e6 into main Apr 24, 2026
10 checks passed
@likp likp deleted the feature/new-metaid-in-json2-serializer branch April 24, 2026 13:49
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.

3 participants