Skip to content

Conversation

@Adnan-cds
Copy link
Contributor

@Adnan-cds Adnan-cds commented Mar 5, 2024

  • Restricts address lookup to a local authority. This is an optional feature.
  • Makes the field label of the address lookup element visible by default. Like all Webform composite elements, it had the label hidden by default.
  • Liberty Create low-code platform integration example.
  • Coding standards fixes.

Adnan-cds and others added 22 commits December 13, 2023 11:04
The local custodian code is a numeric code used by the OS Places API.  Every
local authority has been assigned a code.

Our address lookup element now applies the local custodian code to all Geocoder
plugins it is using.  Those plugins that understand this code then applies it to
the address lookup process.
Not all Geocoder providers are configurable.  These cannot accept the
local_custodian_code parameter.
Correcting recently introduced config.
New tokens and pseudo-tokens:
- [webform_submission:values:FORM-FIELD-ID:extracted_firstname]
- [webform_submission:values:FORM-FIELD-ID:extracted_lastname]
- [webform_submission:values:FORM-FIELD-ID:file_details_for_liberty_create_api]
- [webform_submission:purge_date]
Submodule containing Liberty Create API integrated Webform.
These tokens are necessary for Liberty Create API integration.
Fix running tests with Paratest in Github Actions
We want to display the title of the Address lookup element by default.  At the
moment, the relevant configuration chooses "Invisible" title display by default.
…egration-example

Feature: Liberty create integration essentials
We are providing the Local custodian code through a GeocodeQuery rather than
individually passing it to each geocoder plugin.  This is a cleaner approach
made possible by Ekes' recent change to the geocoder contrib
module: https://www.drupal.org/project/geocoder/issues/3406303
…ion' into feature/borough-address-restriction
…default-config-override

Feature: Display Adddress lookup element title by default
Copy link
Contributor

@andybroomfield andybroomfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to release, thanks @Adnan-cds

@Adnan-cds
Copy link
Contributor Author

Thanks Andy. Merging...

@Adnan-cds Adnan-cds merged commit 044fc79 into release-1.x Mar 6, 2024
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.

5 participants