Skip to content

Added recent changes and moved byIdAndDate to new profile#565

Open
alexgordtop wants to merge 13 commits intoSeBa/get-aas-and-sm-version-by-id-and-datefrom
AlGo/recent-changes-and-history
Open

Added recent changes and moved byIdAndDate to new profile#565
alexgordtop wants to merge 13 commits intoSeBa/get-aas-and-sm-version-by-id-and-datefrom
AlGo/recent-changes-and-history

Conversation

@alexgordtop
Copy link
Copy Markdown
Collaborator

No description provided.

Comment on lines +98 to +100
- https://admin-shell.io/aas/API/GetAllAssetAdministrationShells/3/2
- https://admin-shell.io/aas/API/GetAllAssetAdministrationShellsByAssetId/3/2
- https://admin-shell.io/aas/API/GetAllAssetAdministrationShellsByIdShort/3/2
Copy link
Copy Markdown
Collaborator

@BirgitBoss BirgitBoss Mar 30, 2026

Choose a reason for hiding this comment

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

only GetAssetAdministrationShellVersionByIdAndDate? Probably Get AAS Version by ID or AssetId or Date

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

GetAllAssetAdministrationShellsByAssetId and GetAllAssetAdministrationShells look also wrong to me... Maybe a copy-paste issue?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Should have been:

  x-semanticIds:
    - https://admin-shell.io/aas/API/GetAssetAdministrationShellRecentChanges/3/2

|createdAt |The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
|updatedAt |The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
e|cursor a|The position from which to resume a result listing |string |1
e|id |Globally unique identification of the Asset Administration Shell |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.1/spec-metamodel/datatypes.html#Identifier[Identifier] |1
Copy link
Copy Markdown
Collaborator

@BirgitBoss BirgitBoss Mar 30, 2026

Choose a reason for hiding this comment

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

Suggested change
e|id |Globally unique identification of the Asset Administration Shell |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.1/spec-metamodel/datatypes.html#Identifier[Identifier] |1
e|id a|Globally unique identification of the Asset Administration Shell |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#Identifier[Identifier] |1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Are you assuming the updatedAt information or version information is somehow part of the AAS ID?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I wouldn't enforce that. Versioned IDs have turned out to be a pretty bad idea, they make it hard to find the same object over time.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.1.2/spec-metamodel/common.html#administrative-information-attributes we write:

Note 1: typically, some of the administrative information like the version number might be part of the identification (Submodel/id). This is similar to the handling of identifiers for concept descriptions using IRDIs. In ECLASS, the IRDI 0173-1#02-AO677#002 contains the version information 002.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think I mixed recentChanges and dateAndTime

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I wouldn't enforce that. Versioned IDs have turned out to be a pretty bad idea, they make it hard to find the same

I agree

But this means that the AAS and Submodel repository do not contain archived versions (with same ID), this is an additional component with different APIs.

h|semanticId 3+| `\https://admin-shell.io/aas/API/DataTypes/RecentChange/3/2`

h|Attribute h|Explanation h|Type h|Card.
|createdAt |The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
|createdAt |The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
|createdAt a|The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1


h|Attribute h|Explanation h|Type h|Card.
|createdAt |The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
|updatedAt |The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
|updatedAt |The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
|updatedAt a|The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1

|updatedAt |The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
e|cursor a|The position from which to resume a result listing |string |1
e|id |Globally unique identification of the Asset Administration Shell |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.1/spec-metamodel/datatypes.html#Identifier[Identifier] |1
e|id a|Globally unique identification of the Submodel |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.1/spec-metamodel/datatypes.html#Identifier[Identifier] |1
Copy link
Copy Markdown
Collaborator

@BirgitBoss BirgitBoss Mar 30, 2026

Choose a reason for hiding this comment

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

Only one Submodel? So no result for AAS only or for a single AAS only or for an AAS together with all its Submodels?

Probabably both optional?

You cannot have twice the same attribute "id". Do you mean it depends on context which ID is returned? ==> remove both IDs here, they are added later

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

But same for the AAS.id in the row above, or? AssetAdministrationShellRecentChange and SubmodelRecentChange include them.

Comment thread Part2-API-Schemas/openapi.yaml
Comment thread Part2-API-Schemas/openapi.yaml Outdated
UpdatedAt:
name: updatedAt
in: query
description: Date and time when the Identifiable earliest was created in [ISO 8601](https://www.iso.org/iso-8601-date-and-time-format.html) format.
Copy link
Copy Markdown
Collaborator

@BirgitBoss BirgitBoss Mar 30, 2026

Choose a reason for hiding this comment

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

why add "earliest"? We need "create before or at exactly at this date" Earliest means the opposite: created after or at exactly this date?

Comment thread documentation/IDTA-01002-3/modules/ROOT/pages/specification/interfaces.adoc Outdated
h|Attribute h|Explanation h|Type h|Card.
|createdAt |The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
|updatedAt |The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
e|cursor a|The position from which to resume a result listing |string |1
Copy link
Copy Markdown
Collaborator

@BirgitBoss BirgitBoss Mar 30, 2026

Choose a reason for hiding this comment

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

if there is a cursor there shall also be a limi: pagination should be identical in all cases it is used

However, in other API operation cursor and limit are input parameters and not part of the payload class: this seems inconsistent to the rest?


h|Attribute h|Explanation h|Type h|Card.
|createdAt |The point in time at which the Identifiable object was created |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
|updatedAt |The point in time at which the Identifiable object was recently updated |link:https://industrialdigitaltwin.io/aas-specifications/IDTA-01001/v3.2/spec-metamodel/datatypes.html#DateTimeUtc[DateTimeUtc] | 1
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

we could also add version and revision (both optional) to this class, otherwise we would need another one for these attributes

Comment thread AssetAdministrationShellRepositoryServiceSpecification/V3.2_SSP-001.yaml Outdated
Comment thread AssetAdministrationShellRepositoryServiceSpecification/V3.2_SSP-001.yaml Outdated
Comment thread AssetAdministrationShellRepositoryServiceSpecification/V3.2_SSP-004.yaml Outdated
Comment thread AssetAdministrationShellRepositoryServiceSpecification/V3.2_SSP-004.yaml Outdated
Co-authored-by: sebbader-sap <107036549+sebbader-sap@users.noreply.github.com>
Comment thread SubmodelRepositoryServiceSpecification/V3.2_SSP-006.yaml Outdated
Comment thread SubmodelRepositoryServiceSpecification/V3.2_SSP-006.yaml Outdated
Comment thread SubmodelRepositoryServiceSpecification/V3.2_SSP-006.yaml Outdated
@sebbader-sap
Copy link
Copy Markdown
Contributor

Sorry for the chaos in my comments, took me some time to get the idea...

@alexgordtop alexgordtop marked this pull request as ready for review April 2, 2026 11:26
@BirgitBoss BirgitBoss requested a review from Martin187187 April 9, 2026 07:26
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