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

Add support for primary and secondary contact information #250

Merged

Conversation

Windvis
Copy link
Contributor

@Windvis Windvis commented Feb 1, 2022

This PR adds support for adding a secondary telephone number on all pages with contact information or updates the labels of the ones that already supported this.

It expects that the migrations from lblod/app-organization-portal#88 ran already. Once we merge this we can also merge that PR.

@Windvis
Copy link
Contributor Author

Windvis commented Feb 1, 2022

There are a lot of commits since I created a commit for each page. I might reduce this by combining all related pages into one commit?

@Windvis Windvis force-pushed the feature/support-primary-secondary-contact-information branch 2 times, most recently from 4a31d52 to 8e894bd Compare February 1, 2022 15:21
@Windvis Windvis marked this pull request as ready for review February 1, 2022 15:24
@Windvis Windvis force-pushed the feature/support-primary-secondary-contact-information branch from cb8690f to 452a959 Compare February 2, 2022 09:43
@Windvis Windvis requested a review from nbittich February 2, 2022 09:43
… work with it

This type will be used to differentiate between "primary" and "secondary" contact information.
This can be used to request address and contact information for a site, which we do on the administrative unit core data pages and on the site pages.
- Add a secondary phonenumber field
- Update the labels
This can be used to display address and contact information for a site.
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"
- Add a secondary telephone field
- Update the labels to "primair" and "secundair"
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"
This can be used to edit the address and contact information for a position.
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"t
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"t
- Add a secondary phone number field
- Update the labels to "primair" and "secundair"t
This util is replaced by a version that uses the `type` attribute instead.
@nbittich nbittich force-pushed the feature/support-primary-secondary-contact-information branch from 452a959 to 61bca03 Compare February 2, 2022 13:21
Copy link
Collaborator

@nbittich nbittich left a comment

Choose a reason for hiding this comment

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

tested it locally, seems ok. Had to recreate my db for some reason (probably unrelated).
from the code perspective, from what I see it seems good

@nbittich nbittich merged commit 539e5b8 into development Feb 2, 2022
@nbittich nbittich deleted the feature/support-primary-secondary-contact-information branch February 2, 2022 13:58
@Windvis Windvis added the enhancement New feature or request label Feb 4, 2022
nbittich pushed a commit that referenced this pull request Feb 10, 2022
* Add a type attribute to the contact-point model and some utilities to work with it

This type will be used to differentiate between "primary" and "secondary" contact information.

* Create a `<Site::ContactEditCard>` component

This can be used to request address and contact information for a site, which we do on the administrative unit core data pages and on the site pages.

* Update the contact section on the administrative unit creation page

- Add a secondary phonenumber field
- Update the labels

* Update the contact section on the administrative unit edit page

* Create a `<Site::ContactDataCard>` component

This can be used to display address and contact information for a site.

* Update the contact section on the administrative unit details page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"

* Update the contact section on the site creation page

- Add a secondary telephone field
- Update the labels to "primair" and "secundair"

* Update the contact section on the site edit page

* Update the contact section on the site details page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"

* Create a `<Position::ContactEditCard>` component

This can be used to edit the address and contact information for a position.

* Update the contact section of the mandatory creation page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"

* Update the contact section of the mandatory edit page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"

* Create a `<Position::ContactDataCard>` component

* Update the contact section of the mandatory details page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"

* Allow passing in `@model` and `@models` arguments to the edit button component

* Link to the mandatory edit page from the position details page

* Update the contact section on the minister creation page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"t

* Update the contact section on the minister edit page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"t

* Update the contact section on the minister details page

- Add a secondary phone number field
- Update the labels to "primair" and "secundair"t

* Remove the previous `findPrimaryContact` util

This util is replaced by a version that uses the `type` attribute instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants