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

Lmb 420/entity to add email to bestuursorgaan #133

Merged
merged 10 commits into from
May 13, 2024

Conversation

JonasVanHoof
Copy link
Contributor

@JonasVanHoof JonasVanHoof commented May 7, 2024

Description

We want a user to add an email to a bestuurseenheid. We create a new entity for this and set an inverse relation on the bestuurseenheid. The user will fill in a form to update this email so we also create a form for this.

How to test

  1. Run this backend together with the frontend and follow the instructions in the frontend PR to test it

Links to other PR's

@JonasVanHoof JonasVanHoof self-assigned this May 7, 2024
config/dispatcher/dispatcher.ex Outdated Show resolved Hide resolved
config/form-content/bestuurseenheid-contact/form.ttl Outdated Show resolved Hide resolved
config/form-content/bestuurseenheid-contact/form.ttl Outdated Show resolved Hide resolved
@JonasVanHoof JonasVanHoof force-pushed the LMB-420/entity-to-add-email-to-bestuursorgaan branch from 54e3b05 to 0ab1be1 Compare May 8, 2024 09:51
@JonasVanHoof JonasVanHoof marked this pull request as ready for review May 8, 2024 09:54
config/dispatcher/dispatcher.ex Outdated Show resolved Hide resolved
config/resources/external-besluit.lisp Outdated Show resolved Hide resolved
@JonasVanHoof JonasVanHoof force-pushed the LMB-420/entity-to-add-email-to-bestuursorgaan branch from bc96fdc to d6cfcac Compare May 13, 2024 07:31
@JonasVanHoof JonasVanHoof requested a review from Rahien May 13, 2024 07:32
@JonasVanHoof JonasVanHoof merged commit f474b4d into master May 13, 2024
@JonasVanHoof JonasVanHoof deleted the LMB-420/entity-to-add-email-to-bestuursorgaan branch May 13, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants