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

BTRIA-2353 - Fix for WPML Integration with Search Location #4693

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

sdokus
Copy link
Contributor

@sdokus sdokus commented Aug 9, 2024

🎫 Ticket

BTRIA-2353

🗒️ Description

Grabbing the change from #4603 this will fix the issue where WPML was breaking the location search from Google Maps API. All that was missing was the list of element ids to be merged into the array returned from translate_elements(). Credit goes to @dgwatkins !

🎥 Artifacts

Uploading CleanShot 2024-08-09 at 16.32.39.mp4…

✔️ Checklist

  • Ran npm run changelog to add changelog file(s).
  • Code is covered by NEW wpunit or integration tests.
  • Code is covered by EXISTING wpunit or integration tests.
  • Are all the required tests passing?
  • Automated code review comments are addressed.
  • Have you added Artifacts?
  • Check the base branch for your PR.
  • Add your PR to the project board for the release.

@sdokus sdokus added the code review Status: requires a code review. label Aug 9, 2024
@sdokus sdokus self-assigned this Aug 9, 2024
@sdokus sdokus requested review from bordoni, Camwyn, redscar, dpanta94 and a team and removed request for bordoni, Camwyn, redscar and dpanta94 August 12, 2024 15:16
@sdokus sdokus added the hold Status: on hold–do not proceed with other status items. label Aug 12, 2024
@sdokus
Copy link
Contributor Author

sdokus commented Aug 12, 2024

@andrasguseo is going to do some recon with WPML team for how to recreate consistently for the QA team

@sdokus sdokus removed the hold Status: on hold–do not proceed with other status items. label Aug 19, 2024
Camwyn
Camwyn previously approved these changes Aug 21, 2024
@Camwyn
Copy link
Member

Camwyn commented Aug 21, 2024

@sdokus the linked ticket is still in triage and not in the Drac project - you should reach out to Emily to get it moved. Before merging in case we need to bump it.

@Camwyn Camwyn merged commit 324925d into release/T24.drac Aug 22, 2024
25 of 28 checks passed
@Camwyn Camwyn deleted the fix/btria-2353-wpml-location-search branch August 22, 2024 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code review Status: requires a code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants