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

Release v1.6/hulpmiddelen #302

Merged
merged 112 commits into from
Nov 13, 2024
Merged

Release v1.6/hulpmiddelen #302

merged 112 commits into from
Nov 13, 2024

Conversation

ruthkoole
Copy link
Collaborator

@ruthkoole ruthkoole commented Oct 1, 2024 •
edited
Loading

Beschrijf jouw aanpassingen

  • hernoem instrumenten naar hulpmiddelen
  • voeg overzicht toe van hulpmiddelen
  • voeg functionaliteit toe om categorie toe te voegen aan een hulpmiddel

Bij welk issue hoort deze pull-request?

#314, #317, #90, #227

Checklist before requesting a review

  • Ik heb de contributing guidelines van deze repository gelezen en gevolgd.
  • Ik heb mijn aanpassingen gecheckt op spelfouten.
  • Als ik gebruik heb gemaakt van links, dan heb ik gecheckt of deze werken.
  • Ik heb gebruik gemaakt van de templates en formats van het algoritmekader.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 1, 2024 •
edited
Loading

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-11-13 10:41 UTC

@ruthkoole
Copy link
Collaborator Author

Lekker bezig met de hulpmiddelen!

@ruthkoole ruthkoole force-pushed the release-v1.6/hulpmiddelen branch from c98db95 to a0d284c Compare October 11, 2024 09:04
This was linked to issues Oct 11, 2024
Nieuwe instrumenten/hulpmiddelen toeveogen #317
Closed
ISO normen toevoegen als instrument #298
Closed
Uitgewerkte tekst over standaarden #90
Closed
Overzichtspagina instrumenten updaten #227
Closed
@noutvandeijck noutvandeijck self-assigned this Oct 15, 2024
@noutvandeijck
Update lists.py
c7e3322 
Mogelijk maken dat in maatregelen ook hulpmiddelen getagd kunnen worden, en dat dan op de pagina van een hulpmiddel die maatregelen worden getoond.
@noutvandeijck
Update DPIA.md
bdfe6f5 
Bijbehorende maatregelen bij dit hulpmiddel tonen
@noutvandeijck
Copy link
Collaborator

noutvandeijck commented Oct 15, 2024 •
edited
Loading

@ruthkoole ik probeer mogelijk te maken dat we maatregelen ook aan hulpmiddelen kunnen taggen (net als dat we vereisten aan hulpmiddelen kunnen taggen), en heb daarvoor lists.py en DPIA.md aangepast, maar het werkt nog niet. Zou jij wanneer je tijd hebt eens mee kunnen kijken?

Sterker nog: ook de filters werken niet meer voor maatregelen, ook op de algemene maatregelen-pagina niet (in de preview van dit PR specifiek).

noutvandeijck and others added 10 commits November 4, 2024 16:42
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/handreiking-non-discriminatie.md
3700239 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/inkoopvoorwaarden.md
a5d3f8f 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/inkoopvoorwaarden.md
ad9a4d4 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/mensenrechtelijktoetsingskader.md
bb5c169 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/mensenrechtelijktoetsingskader.md
198ac75 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/onderzoekskader-ADR.md
d09fce6 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/onderzoekskader-ADR.md
cc173dd 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck @ruthkoole
Update docs/hulpmiddelen/onderzoekskader-ADR.md
3511090 
Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck
Update DEDA.md
2686786 
Stuk over relevantie van DEDA toegevoegd.
@giuliaboss
Update BIO.md
afde4a0
@ruthkoole ruthkoole removed a link to an issue Nov 8, 2024
ISO normen toevoegen als instrument #298
Closed
ruthkoole
ruthkoole commented Nov 8, 2024
View reviewed changes
docs/hulpmiddelen/Toetsingskader.md Outdated Show resolved Hide resolved
docs/hulpmiddelen/BIO.md Outdated Show resolved Hide resolved
docs/hulpmiddelen/onderzoekskader-ADR.md Outdated Show resolved Hide resolved
ruthkoole and others added 4 commits November 8, 2024 14:39
@ruthkoole
Update docs/hulpmiddelen/onderzoekskader-ADR.md
461dc89
@ruthkoole
fix links
11b582a
@noutvandeijck @ruthkoole
Apply suggestions from code review
0d3dcaa 
Goede suggesties voor links, doorgevoerd.

Co-authored-by: Ruth Koole <71120805+ruthkoole@users.noreply.github.com>
@noutvandeijck
Update framework-meaningful-engagement.md
3e1832a 
categorie toegevoegd
@noutvandeijck
Copy link
Collaborator

Ik krijg de filters ook niet zo snel werkend. Wellicht kan @rubenrouwhof meekijken? Op de pagina van hulpmiddelen hebben we proberen filters toe te voegen, maar ze doen het nog niet.

Behalve dit, staat alles goed. Dus zodra gefixt is, kan deze PR approved worden.

@ruthkoole
Copy link
Collaborator Author

Ik heb de filters nu uitgezet op de pagina voor hulpmiddelen. Nog kijken hoe we deze functionaliteit kunnen verbeteren

@ruthkoole ruthkoole mentioned this pull request Nov 13, 2024
Open
noutvandeijck
noutvandeijck approved these changes Nov 13, 2024
View reviewed changes
Copy link
Collaborator

@noutvandeijck noutvandeijck left a comment

Choose a reason for hiding this comment

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

helemaal goed zo!

@ruthkoole ruthkoole merged commit 9d45c60 into main Nov 13, 2024
1 check passed
@ruthkoole ruthkoole deleted the release-v1.6/hulpmiddelen branch November 13, 2024 10:41
@ruthkoole ruthkoole mentioned this pull request Nov 13, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@noutvandeijck noutvandeijck noutvandeijck approved these changes

@jaspervanderheide jaspervanderheide Awaiting requested review from jaspervanderheide

Assignees

@giuliaboss giuliaboss

@noutvandeijck noutvandeijck

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@ruthkoole @noutvandeijck @giuliaboss