Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adapter.http: implement AAS API submodel routes via redirects #27

Merged
merged 1 commit into from
Mar 31, 2024

Conversation

jkhsjdhjs
Copy link

The submodel routes of the AAS API /shells/<aas_id>/submodels are the same already implemented as the submodel API, so we return a redirect here, after checking that the AAS indeed has a reference to the requested submodel.

PUT and DELETE are different, as we also have to update the AAS in this case, either by adding a new updated reference in case of a PUT request changes the submodel id, or by removing the submodel reference for DELETE requests.

The submodel routes of the AAS API `/shells/<aas_id>/submodels` are the
same already implemented as the submodel API, so we return a redirect
here, after checking that the AAS indeed has a reference to the
requested submodel.

`PUT` and `DELETE` are different, as we also have to update the AAS in
this case, either by adding a new updated reference in case of a `PUT`
request changes the submodel id, or by removing the submodel reference
for `DELETE` requests.
@s-heppner
Copy link

LGTM

@Frosty2500
Copy link
Collaborator

Also LGTM

@jkhsjdhjs jkhsjdhjs merged commit 1f2a4a5 into feature/http_api Mar 31, 2024
14 checks passed
@jkhsjdhjs jkhsjdhjs deleted the http_api/add_aas_submodel_routes branch April 1, 2024 20:19
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