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

Merge people script should allow merging people who look different, but require a confirmation #234

Open
jfly opened this issue Nov 19, 2015 · 2 comments

Comments

@jfly
Copy link
Contributor

jfly commented Nov 19, 2015

(Suggested by @SAuroux here)

This will let us get rid of the javascript blocking dialog for confirmation, too.

@SAuroux
Copy link
Member

SAuroux commented Nov 25, 2015

After first practically using this script I have a slightly different proposal:

  1. Display the information for both people: IDs, names, countries, DOBs
  2. Let the user select each individual piece of data for the merge (if different):

For example, I just had to merge two people with same countryIDs, but the remaining data being different. I needed to end up with the following: ID1, name2, DOB2, (country)

@jfly
Copy link
Contributor Author

jfly commented Nov 25, 2015

This is feeling more and more like the edit/change person script. I'm now wondering if we can combine the 2 in a useful way...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

3 participants