Skip to content
New issue

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

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

Already on GitHub? # to your account

Update from Working-model: bc3df0f #135

Closed
wants to merge 3 commits into from

Conversation

CX-Machine-User
Copy link

@CX-Machine-User CX-Machine-User commented Oct 21, 2024

This PR updates the branch with the latest changes from Working-model. Commit ID: bc3df0f

These are the last changes from working model. Would like to merge them, before we execute the versioning plug-in.

@ther3sa and @werner-roman you are the only once who are left for review :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Not readable in dark mode - needs background

@@ -1,16 +1,15 @@
---
sidebar_position: 5
title: Catena-X office
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should keep the h1 heading - as this is used by all the other documents as well - at leas I don't see a benefit in using title?

Copy link
Contributor

Choose a reason for hiding this comment

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

we are usually using title instead of h1

Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a benefit? otherwise i think we should keep it consistent across all documents?

@@ -33,7 +33,7 @@ There are two main organizations in GitHub:

:::note

- for the open-source part (reference implementations and Kits) GitHub is mandatory. Its used for planning, issue tracking, discussions, development, reviews and release etc.
- for the open-source part (reference implementations and KITs) GitHub is mandatory. Its used for planning, issue tracking, discussions, development, reviews and release etc.
- for the standardization part, GitHub is also set as mandatory
- for Committee work and Expert Groups the Association provides private repositories within the *catenax-eV* GitHub organization
Copy link
Contributor

Choose a reason for hiding this comment

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

link ti GitHub org?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not readable in dark mode - needs background

Copy link
Contributor

Choose a reason for hiding this comment

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

Not readable in dark mode - needs background

@jSchuetz88
Copy link
Member

jSchuetz88 commented Nov 14, 2024

Why is this PR empty? is this still up-to-date? Can we close this PR?

@stephanbcbauer
Copy link
Member

since this was never merged, should we delete this PR and create a new one? -> i think the comments by @werner-roman are still valid ;) @Grand-Thibault

@stephanbcbauer
Copy link
Member

@Grand-Thibault would like to delete this PR, since the changes are old, but I still don't want to lose the comments by @werner-roman. @Grand-Thibault, can you please fix it in the working-model?

@jSchuetz88
Copy link
Member

jSchuetz88 commented Mar 12, 2025

It seems, this PR is outdated and not needed anymore. All the contents have been merged within an other PR.

@jSchuetz88 jSchuetz88 closed this Mar 12, 2025
@jSchuetz88 jSchuetz88 deleted the update-from-working-model-bc3df0f branch March 19, 2025 19:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants