Skip to content

Fixes wrong thumbnail field in asset-information#215

Merged
aaronzi merged 1 commit intoeclipse-basyx:mainfrom
aaronzi:main
Mar 12, 2026
Merged

Fixes wrong thumbnail field in asset-information#215
aaronzi merged 1 commit intoeclipse-basyx:mainfrom
aaronzi:main

Conversation

@aaronzi
Copy link
Copy Markdown
Member

@aaronzi aaronzi commented Mar 12, 2026

This pull request updates the handling and naming of thumbnail information within asset information structures in both the codebase and the integration tests. The main change is the renaming of the thumbnail field to defaultThumbnail for conformance to the spec.

Thumbnail field renaming:

  • The field thumbnail in the assetInformation map is renamed to defaultThumbnail in the output of both getAssetAdministrationShellMapByDBID and getAssetAdministrationShellMapsByDBID functions in aas_database.go. [1] [2]

Test updates for new field name:

  • All relevant integration tests in aasrepository_integration_test.go are updated to expect assetInformation.defaultThumbnail instead of assetInformation.thumbnail, including test names and assertions. [1] [2] [3] [4]

@aaronzi aaronzi merged commit 58ea5fd into eclipse-basyx:main Mar 12, 2026
23 checks passed
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.

1 participant