Skip to content

MMSUP-61: Fixed issue with inconsistent feature history versions for composite spaces#1481

Merged
roegi merged 3 commits intomasterfrom
fixFeatureHistoryResponse
Mar 24, 2025
Merged

MMSUP-61: Fixed issue with inconsistent feature history versions for composite spaces#1481
roegi merged 3 commits intomasterfrom
fixFeatureHistoryResponse

Conversation

@roegi
Copy link
Contributor

@roegi roegi commented Mar 14, 2025

  • Also fix API response type to be always FEATURE_COLLECTION when requesting multiple versions of a feature
  • Updated existing tests accordingly
  • Add test to verify version 0 is always used for all features originating from the space that has been extended. If being overridden in the extension space, the default version-numbering system of extension space is used.

@roegi roegi requested a review from qGYdXbY2 March 14, 2025 17:18
@roegi roegi requested a review from a team as a code owner March 14, 2025 17:18
@roegi roegi force-pushed the fixFeatureHistoryResponse branch from 54108b3 to feaa8cf Compare March 17, 2025 11:18
roegi added 3 commits March 19, 2025 10:56
Signed-off-by: Benjamin Rögner <benjamin.roegner@here.com>
Signed-off-by: Benjamin Rögner <benjamin.roegner@here.com>
…composite spaces

- Also fix API response type to be always FEATURE_COLLECTION when requesting multiple versions of a feature
- Updated existing tests accordingly
- Add test to verify version 0 is always used for all features originating from the space that has been extended. If being overridden in the extension space, the default version-numbering system of extension space is used.

Signed-off-by: Benjamin Rögner <benjamin.roegner@here.com>
@roegi roegi force-pushed the fixFeatureHistoryResponse branch from feaa8cf to 7972fa5 Compare March 24, 2025 17:03
@roegi roegi merged commit 5ac7663 into master Mar 24, 2025
3 checks passed
@roegi roegi deleted the fixFeatureHistoryResponse branch March 24, 2025 17:25
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