Added recent changes and moved byIdAndDate to new profile#565
Added recent changes and moved byIdAndDate to new profile#565alexgordtop wants to merge 13 commits intoSeBa/get-aas-and-sm-version-by-id-and-datefrom
Conversation
| - 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 |
There was a problem hiding this comment.
only GetAssetAdministrationShellVersionByIdAndDate? Probably Get AAS Version by ID or AssetId or Date
There was a problem hiding this comment.
GetAllAssetAdministrationShellsByAssetId and GetAllAssetAdministrationShells look also wrong to me... Maybe a copy-paste issue?
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
| 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 |
There was a problem hiding this comment.
Are you assuming the updatedAt information or version information is somehow part of the AAS ID?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
I think I mixed recentChanges and dateAndTime
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
| |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 |
There was a problem hiding this comment.
| |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 |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
But same for the AAS.id in the row above, or? AssetAdministrationShellRecentChange and SubmodelRecentChange include them.
| 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. |
There was a problem hiding this comment.
why add "earliest"? We need "create before or at exactly at this date" Earliest means the opposite: created after or at exactly this date?
| 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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
we could also add version and revision (both optional) to this class, otherwise we would need another one for these attributes
Co-authored-by: sebbader-sap <107036549+sebbader-sap@users.noreply.github.com>
|
Sorry for the chaos in my comments, took me some time to get the idea... |
…etSubmodelRecentChanges
…nd changed description
No description provided.