-
Notifications
You must be signed in to change notification settings - Fork 22
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 two records into a single record #2855
Comments
To implement the merge functionality this is my approach:
Let me know if it looks good to you and if you have any suggestions |
@george-neha, I've created the first draft of the merge screen. You can check it out (https://pr-2879.aam-digital.net/) by navigating to Bulk Action -> Merge. I'm also attaching a screenshot of the current UI. Let me know if you have any feedback or suggestions. In the meantime, I'll continue working on the functionality. |
@sleidig |
That's a good question, @Abhinegi2 ... I think updating one of the existing records is good. It means we have to make less changes to related records (other entities that reference this only have to be changed if they point to the second, deleted ID - although this is only part of a second step #2858 ). At the moment I can't think of a disadvantage of this compared to creating a completely new record. Do you see any potential issues? |
Currently, the only disadvantage I see with creating a completely new record is that the related records from any of the selected entities do not merge. However, when merging into an existing record, we at least retain the references from the selected record(which we are going to cover in 2858). |
@Abhinegi2 In notes, when merging two records, the system displays an internal student ID instead of the child's name. ![]() |
@Abhinegi2 for this as well as some similar remarks: I would suggest to use |
I have integrated the EnfityFieldViewComponent for now, and it seems to be working fine. @george-neha please check now also currently after merging the Notes record related entities not working as expected which I will covering in #2858 |
This is not trivial (e.g. the same field can be displayed under multiple tabs; some fields may not be shown in any tab but still contain old data that has to be merged to avoid data loss; etc.). Let's discuss this separately as a sub-issue to the overall feature |
@george-neha I have added a confirmation dialog similar to other bulk actions, Please check now |
Looks good so far! I have a few extra points on my UX wishlist still:
@george-neha , are these part of any of the other sub-issues already? |
As a user/Social Worker (SW), I want to merge two similar records in an entity into one to avoid duplicates and ensure consistency and accuracy.
Objective
In the system, two users/SWs may record notes/tasks/schools/children twice, which can lead to duplication of work and waste of their time. To avoid this, there should be an option to merge two records of a single record. After merging the two records in the system, there should be only one record with the details captured from both records to ensure no loss of unique data.
Proposed Solutions & Alternatives
Acceptance Criteria:
Exceptions:
Out of scope
Low fidelity skteches for merging two records
The text was updated successfully, but these errors were encountered: