Skip to content

Conversation

@JeremySkinner
Copy link

Initial work on the deduplication UI. The UI has been ported from the D7 opencrm_dedupe module.

Highlights:

  • Implements the merge UI using the D8 form and entity APIs
  • Uses events instead of hooks to register fields for display (see GetPropertiesToShowOnDedupScreenEvent)
  • Modifies the DashboardNegotiator so that the contacts theme doesn't always have to be applied (this is still the default behaviour, but can now be overridden in the route)

Next steps:

  • This needs wiring up to the actual deduplication logic in the entity_merge module (not yet ported)
  • This could possibly move to a separate module
  • Needs a sensible set of default fields (currently uses mail and gender for testing)

@JeremySkinner JeremySkinner requested a review from yanniboi March 7, 2018 15:22
@yanniboi
Copy link
Collaborator

yanniboi commented Mar 7, 2018

Great stuff @JeremySkinner a solid start! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants