@rajeshreeputra This one also looks good to be merged.
Hey folks,
I wanted to keep things simple and straightforward with this approach. I've created a global configuration form for the gmap api key and I've implemented a library info alter hook to add the script if the api key is configured. (Note: whenever you save the configuration form, it will trigger a cache clear)
We currently have two open merge requests, and once either of them gets merged, I'll make the necessary adjustments accordingly. (see: https://www.drupal.org/project/webform_gmap_field/issues/3282421 💬 Tokens for Webforms Fixed )
vzsigmond → made their first commit to this issue’s fork.
This is now outdated and is covered by the new release.
This is now outdated and is covered by the new release.
Hey folks,
I've gone ahead and created an issue fork for this. Here's a merge request for you to check out: https://git.drupalcode.org/project/webform_gmap_field/-/merge_requests/3...
The main thing I've done is switch the element to a composite structure instead of a simple one. This change allows us to easily access the `lat` and `lng` properties.
To keep things smooth, I've added an update hook that handles the migration of existing submissions to the new composite structure.
Take a look and let me know what you think.
Cheers!
vzsigmond → made their first commit to this issue’s fork.