#2841 - Record Merging - Merging Locations Fields Not Working#2870
Merged
corsacca merged 6 commits intoFeb 24, 2026
Conversation
Collaborator
kodinkat
commented
Jan 23, 2026
kodinkat
commented
- fixes: Record Merging, merging locations fields not workings #2841
… field types, improve location_meta handling with JSON parsing, and ensure proper event dispatching for web components. Refactor value updates and deletions for better synchronization with UI components.
Collaborator
Author
|
@corsacca review ready... |
Member
|
It seems like i still have the issue. 
With locations, tags, connections, etc |
…ools-theme into 2841-record-merging-merging-locations-fields-not-working
- Updated primary field selection to ensure they are always checked as the baseline, improving clarity in the Updated column.
- Removed previous auto-select logic for archiving fields to enforce explicit user selection.
- Added timestamp handling for date fields to ensure accurate merging and UI updates.
- Improved initialization timing with setTimeout to ensure all fields are properly set before triggering change events.
Collaborator
Author
|
@corsacca see latest updates.... 
|
Member
|
@kodinkat |
…ools-theme into 2841-record-merging-merging-locations-fields-not-working
- Implemented auto-selection of archiving checkboxes when a corresponding record has a value, improving user experience by ensuring relevant fields are pre-checked.
- Added handling for location_meta fields to correctly parse and validate JSON string values, ensuring accurate data representation.
- Removed outdated auto-select logic for archiving fields to maintain clarity in user selections.
…sistency
- Simplified conditional checks for archiving checkbox selection to enhance clarity.
- Reformatted code for better readability, particularly in the handling of location_meta fields and JSON parsing.
- Ensured consistent indentation and spacing throughout the file to maintain coding standards.
Collaborator
Author
|
@corsacca see latest updates..... 
|
Member
|
Thanks @kodinkat! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.