Skip to content

Added Table-Stateful & Table-Stateless #250

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

Merged
merged 21 commits into from
Nov 23, 2024

Conversation

AlexanderSchmidtCE
Copy link
Contributor

@AlexanderSchmidtCE AlexanderSchmidtCE commented Oct 30, 2024

The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

  • Maturation completed and documented
  • Meaningful pull request title for the release notes
  • Pull request is linked to an issue
  • All changes relate to the issue
  • No TODOs or commented out code in the final commit
  • Tests to protect this code implemented (if applicable)
  • Manual test performed successfully (if applicable)
  • Documentation or migration has been updated (if applicable)

@AlexanderSchmidtCE AlexanderSchmidtCE linked an issue Oct 30, 2024 that may be closed by this pull request
Copy link
Contributor

@deleonio deleonio left a comment

Choose a reason for hiding this comment

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

Wir brauchen das bitte noch auf English. In diesem Verzeichnis noch auf Englisch machen: https://github.com/public-ui/public-ui.github.io/tree/main/i18n/en/docusaurus-plugin-content-docs/current/30-components

@sdvg Kannst Du Alex das bitte erklären.

Es ist korrekt, dass wir aktuell noch keine englische Doku bei den Komponenten haben. Wir haben aber für alle Komponenten eine englische mdx die dann die readme.md auf Deutsch importiert.

@sdvg
Copy link
Member

sdvg commented Nov 11, 2024

@deleonio

Wir haben aber für alle Komponenten eine englische mdx die dann die readme.md auf Deutsch importiert.

Mit der Erklärung hat es für mich Sinn ergeben, das scheint mir aber nicht der Fall zu sein. Die Table wäre tatsächlich die erste Datei in 30-components, neben der index-Datei. Es werden automatisch die deutschsprachigen Seiten angezeigt, wenn keine Übersetzung existiert.

Aus Gründen der Konsistenz würde ich die Datei lieber wieder entfernen, wenn niemand Veto einlegt @AlexanderSchmidtCE.

Copy link

github-actions bot commented Nov 11, 2024

deleonio

This comment was marked as off-topic.

@deleonio deleonio requested review from deleonio and removed request for deleonio November 16, 2024 09:05
Copy link
Contributor

@deleonio deleonio left a comment

Choose a reason for hiding this comment

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

Ich möchte, dass jede Komponente einen eigenen Eintrag in der Menü-Leiste hat. Nicht alles unter Table verstecken.

@AlexanderSchmidtCE bitte hochpriorisieren. Danke

@deleonio deleonio merged commit 5453a44 into main Nov 23, 2024
2 of 3 checks passed
@deleonio deleonio deleted the 242-kachel-fr-die-tabelle-in-der-dokumentation branch November 23, 2024 10:40
@github-actions github-actions bot locked and limited conversation to collaborators Nov 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kachel für die Tabelle in der Dokumentation
4 participants