Skip to content
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

docs: definition of done improvements #775

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions docs/componenten/definition-of-done.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,10 @@ Dit component bestaat nog niet, maar de rationale is duidelijk, elke organisatie
- Staat in de [publieke Backlog](https://github.com/nl-design-system/backlog/issues) van NL Design System.
- Rationale opgesteld voor naam en doel.
- Screenshots verzameld voor diverse huisstijlen, van alle belangrijke varianten en states.
- Varianten van het component die regelmatig voorkomen opgeschreven en beschreven wanneer je ze gebruikt.
- Varianten van het component opgeschreven en beschreven wanneer je ze gebruikt.
- Kernteam en community verwachten dat het component voor meerdere organisaties inzetbaar kan zijn.
- Rationale is onderbouwd door gebruikersonderzoek (eigen of van anderen).
- Doel uit de rationale is duidelijk onderscheidend van andere componenten.
- Rationale is door het kernteam bevestigd.
- Vindbaar op de NL Design System website.

Expand All @@ -46,7 +47,9 @@ Bekijk de status op [het projectbord voor de Help wanted componenten](https://gi
Dit component bestaat in de community, op één of meer plekken. Om bij NL Design System als 'Community component' genoemd te worden moet het component naast alle 'Help wanted' criteria aan de volgende voorwaarden voldoen:

- Implementatie van 1 of meer varianten voldoen aan de rationale.
- Component is publiek beschikbaar.
- Code is publiek beschikbaar via Git.
Robbert marked this conversation as resolved.
Show resolved Hide resolved
- CSS is gepubliceerd in een publieke NPM package.
Robbert marked this conversation as resolved.
Show resolved Hide resolved
- Tellen welke varianten veel gebruikt worden.
Copy link
Member

Choose a reason for hiding this comment

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

‘veel’ is niet concreet, dit item lijkt me sowieso geen geschikt DoD item, zou 'm eruit halen

Copy link
Member Author

Choose a reason for hiding this comment

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

Het doel is "snowflake" varianten eruit halen, dus wel waardevol. Ik zal 'm iets aanscherpen.

- Varianten hebben voldoende design tokens om de huisstijlen uit de verzamelde screenshots mogelijk te maken.
- Component heeft de EUPL-1.2 licentie.
- Documentatie heeft de Creative Commons 0 licentie (CC0).
Expand Down