Skip to content

fix: pass paging_metadata kwarg correctly in get_aas_submodel_refs and get_concept_description_all#540

Merged
s-heppner merged 6 commits into
developfrom
fix/submodel-refs-concept-desc-paging-metadata
May 13, 2026
Merged

fix: pass paging_metadata kwarg correctly in get_aas_submodel_refs and get_concept_description_all#540
s-heppner merged 6 commits into
developfrom
fix/submodel-refs-concept-desc-paging-metadata

Conversation

@zrgt
Copy link
Copy Markdown
Contributor

@zrgt zrgt commented May 12, 2026

Summary

Fixes 5 handler methods across 3 files that called response_t(..., cursor=cursor), but APIResponse.__init__ takes paging_metadata: Optional[PagingMetadata], not cursor. All affected endpoints returned 502 Bad Gateway due to the resulting TypeError.

Affected handlers

File Handler
repository.py get_aas_submodel_refs
repository.py get_concept_description_all
discovery.py get_all_aas_ids_by_asset_link
discovery.py search_all_aas_ids_by_asset_link
registry.py get_all_submodel_descriptors_through_superpath

Closes #539

…d get_concept_description_all

Closes #539

Both handlers called response_t(..., cursor=cursor) but APIResponse.__init__
accepts paging_metadata, not cursor. _get_slice() already returns
Optional[PagingMetadata] as its second value, so the variable just needs
to be passed under the correct keyword argument name.
@zrgt zrgt requested a review from s-heppner May 12, 2026 15:38
@zrgt zrgt requested review from s-heppner and removed request for s-heppner May 13, 2026 09:42
@s-heppner
Copy link
Copy Markdown
Member

Can you fix the static analysis check please?

@s-heppner s-heppner merged commit 10778d0 into develop May 13, 2026
25 checks passed
@s-heppner s-heppner deleted the fix/submodel-refs-concept-desc-paging-metadata branch May 13, 2026 10:54
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