eclipse-tractusx / sldt-semantic-models

sldt-semantic-models
Creative Commons Attribution 4.0 International
22 stars 41 forks source link

[New Model]: Renamed Model: Secondary Material Content #285

Closed catroest closed 7 months ago

catroest commented 9 months ago

Description

SMC Model includes new secondary characteristic attributes. Renamed from Material for Recycling.

Closes #261

MS2 Criteria

(to be filled out by PR reviewer)

MS3 Criteria

(to be filled out by semantic modeling team before merge to main-branch)

catroest commented 9 months ago

@bs-jokri PR on behalf of Sabine and Johannes from SMC

catroest commented 9 months ago

@bs-jokri I don't know if Sabine wrote to you already. But please check this model. This model was renamed from Material for Recycling to Secondary Material Content and changed.

bs-jokri commented 9 months ago

The checks for MS2 where done with the updated model provided with PR https://github.com/eclipse-tractusx/sldt-semantic-models/pull/291

github-actions[bot] commented 9 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

github-actions[bot] commented 8 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

github-actions[bot] commented 8 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

bs-jokri commented 8 months ago

@agg3fe will you have a look?

bs-jokri commented 8 months ago

@catroest @johannsvarela there are still open review comments, who will resolve them? Otherwise we cannot assign MS2

github-actions[bot] commented 8 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

github-actions[bot] commented 8 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

github-actions[bot] commented 8 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

catroest commented 8 months ago

@bs-jokri eventually followed by a deprecation request of the old one

bs-jokri commented 8 months ago

@bs-jokri eventually followed by a deprecation request of the old one

@catroest could you link the deprecation request here?

bs-jokri commented 8 months ago

from discussion property orderNumber will be checked if it should be marked optional.

github-actions[bot] commented 7 months ago

Validation Report for io.catenax.secondary_material_content/1.0.0/SecondaryMaterialContent.ttl

Input model is valid

catroest commented 7 months ago

@bs-jokri ready to merge, order number is now optional

catroest commented 7 months ago

395 old one will be deprecated