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

Filling details with Email not working as expected #85

Open
jaapjansma opened this issue Jul 26, 2021 · 0 comments · May be fixed by #83
Open

Filling details with Email not working as expected #85

jaapjansma opened this issue Jul 26, 2021 · 0 comments · May be fixed by #83
Labels
enhancement status:needs review Code needs review and testing
Milestone

Comments

@jaapjansma
Copy link
Contributor

With the fill details options I would expect that a email is added when no email with the location type exists.

When an email with that location type already exists I would expect that nothing changes. However the new email gets added and I end up with two emails of the same location type.

Steps to reproduce

  1. Create an XCM Profile with no overriding details and at fill details select email, for location type select home
  2. On this profile match on external identifier
  3. Create a contact with external identifier 123aa and add a email [email protected] with location type home
  4. Now use the XCM profile to lookup/create a contact. I have used the form processor for this. Provide data as follows, external identifier 123aa and add the email [email protected] (the email is different)

Expected result

Nothing will happen because an email with the location type home already exists.

Actual result

The email gets added and the contacts has two email with location type home

Remarks

It is possible to have multiple emails with the same location type. So in one sense this behavior is correct.

However in my specific use case this is exactly not what we want.

For the address field we already have such options. Fill if contact has no address, or fill if contact no address of that type.

@jaapjansma jaapjansma linked a pull request Jul 26, 2021 that will close this issue
@jensschuppe jensschuppe linked a pull request Sep 14, 2023 that will close this issue
@jensschuppe jensschuppe added this to the 1.12 milestone Sep 14, 2023
@jensschuppe jensschuppe added enhancement status:needs review Code needs review and testing labels Sep 14, 2023
@bjendres bjendres modified the milestones: 1.12, 1.13 Oct 11, 2023
@jensschuppe jensschuppe modified the milestones: 1.13, 1.14 Jun 25, 2024
@jensschuppe jensschuppe changed the title Filling details with Email number not working as expected Filling details with Email not working as expected Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement status:needs review Code needs review and testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants