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.

Sign up for GitHub

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

Jump to bottom

docs: update community stappenplan #1809

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

docs: update community stappenplan #1809

wants to merge 5 commits into from
+145 −112

Conversation

jeffreylauwers
Copy link
Contributor

@jeffreylauwers jeffreylauwers commented Dec 11, 2024 •
edited
Loading

Community Stappenplan voor kernteam

  • Opzet van schrijfwijze gelijk getrokken omdat dit het Community stappenplan voor het kernteam zelf betreft.
  • Checkpoint 'nldesignsystem' gelijk getrokken met Help Wanted stappenplan.
  • Checkpoint 'promotie' opgesplitst in verband met component progress.
  • Fout in 'Status bijwerken' eruit.
  • Dubbeling van component progress er uit.
  • Tip over minimale tokens toegevoegd.
  • Hier en daar kleine tweaks die mooi meegenomen waren.

Definition of done

  • Heading uit Stappenplan gelijk getrokken met item in opsomming.
  • Positie aangepast.

Community Stappenplan voor community

  • Opzet van schrijfwijze gelijk getrokken omdat dit het Community stappenplan voor de community zelf betreft.
  • Tip over minimale tokens toegevoegd.
  • Hier en daar kleine tweaks die mooi meegenomen waren.

@vercel Vercel
Copy link

vercel bot commented Dec 11, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentatie ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 13, 2024 10:16am

Rozerinay
Rozerinay reviewed Dec 12, 2024
View reviewed changes
docs/handboek/component-bijdragen/community-stappenplan/voor-kernteam.mdx

## Gebruik van component uit de community gepromoot

Doel: Credits geven aan mensen die hebben bijgedragen vanuit de community. De rest van community informeren en enthousiasmeren om ook naar Community, Candidate en Hall of Fame toe te werken.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Doel: Credits geven aan mensen die hebben bijgedragen vanuit de community. De rest van community informeren en enthousiasmeren om ook naar Community, Candidate en Hall of Fame toe te werken.
Doel: Credits geven aan mensen die hebben bijgedragen vanuit de community. De rest van community informeren en enthousiasmeren om ook naar 'Community', 'Candidate' en 'Hall of Fame' statussen toe te werken.

Copy link
Contributor

Choose a reason for hiding this comment

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

We gebruiken soms wel en soms geen aanhalingstekens bij de statussen. Laten we een keuze maken of we dat wel of niet doen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja lastig. Ik hanteer normaal gesproken altijd 1 keer aanhalingstekens als de 'term' voor het eerst wordt geïntroduceerd. En daarna niet meer. Echt iets om een ei over te leggen in de schrijfwijzer.

Rozerinay
Rozerinay reviewed Dec 12, 2024
View reviewed changes
docs/handboek/component-bijdragen/community-stappenplan/voor-kernteam.mdx

## 🏁 Finish

Zo wat een werk! Je hebt alle stappen genomen en je hebt alle checkpoints kunnen afvinken die nodig zijn voor de ['Community' status](https://github.com/orgs/nl-design-system/projects/29/views/1) van het [Estafettemodel](https://www.nldesignsystem.nl/componenten/definition-of-done). De component gaat nu door voor de 'Candidate' status.
Zo wat een werk! Je hebt alle stappen genomen en zo alle checkpoints behaald die nodig zijn voor de ['Community' status](https://github.com/orgs/nl-design-system/projects/29/views/1) van het [Estafettemodel](https://www.nldesignsystem.nl/componenten/definition-of-done). De component gaat nu door voor de 'Candidate' status.
Copy link
Contributor

Choose a reason for hiding this comment

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

Hier plaatsen we bijv. weer aanhalingstekens bij Community en Candidate.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja en dat komt dan weer omdat dit als 'los' bericht in Slack wordt gedeeld.

Rozerinay
Rozerinay reviewed Dec 12, 2024
View reviewed changes
docs/handboek/component-bijdragen/community-stappenplan/voor-organisaties.mdx
1. Open de config.json van het thema van de organisatie
2. Voeg de id van de story of stories toe aan `"stories": []`
1. Open de config.json van het thema van de organisatie.
2. Voeg de id van de story of stories toe aan `"stories": []`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Moet id niet ID zijn?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Weet ik niet. @Yolijn ?

Rozerinay
Rozerinay reviewed Dec 12, 2024
View reviewed changes
docs/handboek/component-bijdragen/community-stappenplan/voor-organisaties.mdx
1. Open de config.json van het Voorbeeld thema `packages/voorbeeld-design-tokens/src/config.json`
2. Voeg de id van de story of stories toe aan `"stories": []`
1. Open de config.json van het Voorbeeld thema `packages/voorbeeld-design-tokens/src/config.json`.
2. Voeg de id van de story of stories toe aan `"stories": []`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Moet id niet ID zijn?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Weet ik niet. @Yolijn ?

Rozerinay
Rozerinay reviewed Dec 12, 2024
View reviewed changes
Copy link
Contributor

@Rozerinay Rozerinay left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Goed werk! Ik ben er doorheen.

We moeten trouwens niet vergeten om de volgorde van de checkpoints op de borden aan te passen.

@jeffreylauwers @Rozerinay
Apply suggestions from code review
2b0d675 
Prachtige toevoegingen Roos! Sterker nog, er zijn er een paar die we ook weer bij Help Wanted moeten doorvoeren denk ik.

Co-authored-by: Rozerin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rozerinay Rozerinay Rozerinay left review comments

@Yolijn Yolijn Awaiting requested review from Yolijn

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jeffreylauwers @Rozerinay