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

#258 fusiegemeenten prep #259

Merged
merged 3 commits into from
Nov 25, 2024
Merged

Conversation

vancamti
Copy link
Contributor

@vancamti vancamti commented Nov 19, 2024

params["status"] = status

filtering zit daar al denk ik voor de rest zie ik niet direct waar het nog zou moeten

@coveralls
Copy link

coveralls commented Nov 19, 2024

Coverage Status

coverage: 53.769% (+0.07%) from 53.7%
when pulling d4ca4d7 on feature/258_add_param_status_gemeenten
into 5622e9f on develop.

@vancamti vancamti force-pushed the feature/258_add_param_status_gemeenten branch from 6a49224 to 722f449 Compare November 20, 2024 13:44
@vancamti vancamti requested a review from claeyswo November 20, 2024 13:52
Copy link
Member

@claeyswo claeyswo left a comment

Choose a reason for hiding this comment

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

Denk hier en hier nog status toevoegen ook.

Copy link
Member

@claeyswo claeyswo left a comment

Choose a reason for hiding this comment

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

Op het einde van de list_gemeenten moet er ook nog filtering komen.

@claeyswo claeyswo merged commit 75a2da2 into develop Nov 25, 2024
5 checks passed
@claeyswo claeyswo deleted the feature/258_add_param_status_gemeenten branch November 25, 2024 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants