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

dev/core#1605 State/province not copied on shared address #16649

Merged
merged 2 commits into from
Mar 5, 2020

Conversation

alifrumin
Copy link
Contributor

Overview

Ensure the state/province is copied when sharing an address.

More details including screenshots in the gitlab issue: https://lab.civicrm.org/dev/core/issues/1605

Before

  1. User goes to a contact's summary tab
  2. Clicks "Add Address"
  3. Selects the "Use another contact's address" checkbox
  4. Selects a contact with an address WITH a state
  5. Clicks Save

Result: All the address fields are copied over EXCEPT the state/province.

After

When sharing an address the state province field IS copied

To Replicate:

  1. User goes to a contact's summary tab
  2. Clicks "Add Address"
  3. Selects the "Use another contact's address" checkbox
  4. Selects a contact with an address WITH a state
  5. Clicks Save

Result: All the address fields are copied over INCLUDING the state/province.

@civibot
Copy link

civibot bot commented Feb 28, 2020

(Standard links)

@civibot civibot bot added the master label Feb 28, 2020
@seamuslee001
Copy link
Contributor

Jenkins re test this please

2 similar comments
@alifrumin
Copy link
Contributor Author

Jenkins re test this please

@alifrumin
Copy link
Contributor Author

Jenkins re test this please

@eileenmcnaughton
Copy link
Contributor

I was just tormenting myself over whether we could get away with this without a test & then I scrolled down & lo-and-behold...

@eileenmcnaughton
Copy link
Contributor

OK - I was able to replicate & see this fix - I did have some weird experiences doing it but they were in the formRule which is untouched so I think unrelated

@eileenmcnaughton eileenmcnaughton merged commit df29259 into civicrm:master Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants