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

Feedback op stap "Voeg de component toe aan een config" in het NLDS Community Stappenplan #2021

Open
2 tasks
eslook opened this issue Feb 17, 2025 · 0 comments · May be fixed by #2022
Open
2 tasks

Feedback op stap "Voeg de component toe aan een config" in het NLDS Community Stappenplan #2021

eslook opened this issue Feb 17, 2025 · 0 comments · May be fixed by #2022
Assignees

Comments

@eslook
Copy link
Contributor

eslook commented Feb 17, 2025

Bij het bijdragen aan het NL Design System door een component van Community naar Candidate te zetten, stuit de community op onduidelijkheden in het stappenplan bij de stap "Voeg de component toe aan een config".

In deze PR kwam de volgende feedback naar voren:

Kan het zijn dat er een stap is overgeslagen, namelijk "Voeg de component toe aan een config"? Ik zie namelijk geen wijzigingen in de config.json van de voorbeeld-design-tokens, noch een config.json voor de Rijkshuisstijl Community. Zie: stap in het handboek.

Originally posted by @eslook in GitHub Issue #942

De community heeft echter voor deze stap al via Slack gevraagd hoe deze stap aan te pakken: "Nu lopen we vast bij de stap waarin het component moet worden toegevoegd aan de theme-toolkit. Hiervoor lijkt een extra stap nodig om het component aan een config toe te voegen.".

Waarop Robbert heeft aangegeven dat dit niet nodig is, hij gaf de volgende aanwijzingen:

  1. Maak een bestand zoals /packages/theme-toolkit/src/component-stories-ams.tsx.
  2. Maak een bestand zoals /packages/voorbeeld-design-tokens/documentation/button/ams-button.stories.tsx.
  3. Voeg je eigen theme toe aan de configuratie in /packages/storybook/config/preview.ts.

Deze feedback moet worden verwerkt in het stappenplan, zodat het correct en volledig is.

Acceptatiecriteria

  • De stap "Voeg de component toe aan een config" is aangepast zodat deze correct en volledig is.
  • Het kernteam heeft het gewijzigde stappenplan gevalideerd.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant