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#2152: Merge contacts keep case roles #18884

Merged
merged 1 commit into from
Oct 30, 2020

Conversation

jaapjansma
Copy link
Contributor

Overview

When you merge contacts and and the old and new contact had both a role on two different cases with the same client. One of the roles gets deleted.

Before

Case roles might get deleted upon merging of contacts (if they have the same client)

After

Case roles are kept upon merging of contacts

Comments

See https://lab.civicrm.org/dev/core/-/issues/2152 for a more detailed explanation.

@civibot
Copy link

civibot bot commented Oct 30, 2020

(Standard links)

@civibot civibot bot added the master label Oct 30, 2020
@demeritcowboy
Copy link
Contributor

  • General standards
    • [r-explain] PASS
      • I'm not sure when it stopped working but I know for a fact this used to work properly some time in the past. I don't know how recent.
    • [r-user] PASS
    • [r-doc] PASS
    • [r-run] PASS
  • Developer standards
    • [r-tech] PASS
      • It looks right. And I think the circular one is right too but it makes my head spin: E.g. Alice1 is the spouse for client Bob (case1) and Bob is the spouse for client Alice1 (case2) and Alice2 is the spouse for client Bob (case3) and Bob is the spouse for client Alice2 (case4). Now merge Alice1 and Alice2. That's kind of a messed up setup to begin with, but the end result looks ok when I try it.
    • [r-code] PASS
    • [r-maint] Iffy (grin)
    • [r-test] PASS

@colemanw colemanw merged commit b6e1f6b into civicrm:master Oct 30, 2020
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