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

#2494 - Merge Location_Grid And Location_Grid_Meta Into A Single Field #2519

Draft
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

kodinkat
Copy link
Contributor


Further Design Discussion Points

  • Need to determine which field should be updated when record contains multiple meta location fields and contact_address field populated.

  • Once proposed shape has been merged; need to identify plugins impacted by changes and in need of updates:

    • DT_Webform
    • DT_Import

@corsacca
Copy link
Member

corsacca commented Jul 12, 2024

@kodinkat If i use the API to set the location_grid to Paris as so:
image

The location field stays empty:
image

On the contact the location grid field acquires 2 values:
image

We'll want to merge these into one value and make sure the field is displayed.
If the user updates the contact via location_grid or location_grid_meta we want to end up with the same result

@kodinkat
Copy link
Contributor Author

403 Forbidden exception encountered whilst trying to directly access API; which I believe is the result of an abscent nonce. As a result, no updates are made to location field.

Screenshot 2024-07-25 at 11 49 45

Recent updates now ensure multiple postmeta location grid entries are no longer made.

…ools-theme into 2494-merge-location_grid-and-location_grid_meta-into-a-single-field
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.

Merge location_grid and location_grid_meta into one field
2 participants