-
Notifications
You must be signed in to change notification settings - Fork 89
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
[Industry Core KIT] doc: added new unique id push notification illustration #1057
[Industry Core KIT] doc: added new unique id push notification illustration #1057
Conversation
docs-kits/kits/Industry Core Kit/Software Development View/part_uniqueidpush.mdx
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@johannsvarela THX for this contribution ⇾ one question related to the picture. Is this just an exported SVG? Or could it also directly be adapted? Question ⇾ where are the sources, if someone from the open-source community want to adapt the picture. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As @stephanbcbauer mentioned,
- please provide the "raw" file like the drawio
- please add a dedicated license file for the svg and raw file (TRG 7.08) -> same location and same name and append
.license
(content as mentioned in the trg with your companies that created the image OR just the contributors to the ef)
Sidequestion to the image: Does it make sense to state the direction in terms of the bom (bom, usage) as we're trying to build / update that semantic model?
@tom-rm-meyer-ISST: Question to your sidequestion: do you want to add an information where the singlelevelbom (=at customer) and where the singlelevelusage (=at supplier) is maintained? Or do you have something else in mind? |
@stephanbcbauer @tom-rm-meyer-ISST unlike png files, this svg file of the image can be inserted and adapted directly into tools such as drwa.io. I would therefore not add an additional raw file. If you still insist, I'm happy to add that, but this file would never be used in KIT, but would only remain in the asset folder as an additional file. |
@tom-rm-meyer-ISST i'm not sure if i get your point of your sidequest? 😅 |
Ah cool, sorry didnt test it if it is possible to open in draw.io -> it works (my concern was, not all SVG are editable in drawio and its also mentioned in the svg file
thx for adding the license file |
docs-kits/kits/Industry Core Kit/assets/unique_id_push_notifications.svg.license
Outdated
Show resolved
Hide resolved
@johannsvarela & @eckardg: my sidequestion was partially stupid as the information is partially available (after downloading the svg I can zoom :D ). When I recently read the IC KIT and this feature, I tried to think "OK which For me personaly I would also think about highlighting the model names (e.g. make them bold). Nevertheless, I added one comment regarding the license year (might be even wrong). Just ping me for the approval afterwards (I'll be available till monday mid of the day). |
…ations.svg.license Co-authored-by: Tom Meyer <[email protected]>
@tom-rm-meyer-ISST done as suggested! 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Let me know, if I should trigger the merge :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Enhancement of the existing documentation for the unique id push notifications with a new illustration to describe the functionality of the notifications in a visual way. For detailed information have a look at the changelog.
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: