Refactor LocalGov Drupal to use https://www.drupal.org/project/geo_entity

[finnlewis]

We now have https://www.drupal.org/project/geo_entity

It would be good to refactor LocalGov Drupal to use the d.o module.

We would need to write an update hook to migrate all existing entities.

• migrate all localgov_geo entities to geo_entity entities
• update all entity reference fields
• add docs to advise switching geocode to OS places

We should really try to do this sooner rather than later. The longer we leave it the more gnarly it gets. Technical debt warning!

[ekes]

Update Drupal 9 https://github.com/localgovdrupal/localgov_project/tree/feature/2.x/geo-entity
Then Drupal 10 https://github.com/localgovdrupal/localgov_project/tree/feature/3.x/geo_entity

Which are checking out and running updates from
The Drupal 9 that just enables an update module https://github.com/localgovdrupal/localgov_geo/tree/feature/1.x/geo-entity
And Drupal 10 https://github.com/localgovdrupal/localgov_geo/tree/feature/2.x/geo_entity that does the update.

[ekes]

Requires test.

Might require also copying over view and form modes.

[ekes]

Might require also copying over view and form modes.

Doing this seems somewhat more involved because of changing from geofield_map to leaflet. Which is the desired change, so the defaults work. It would be if additional fields have been configured, or order. We could try and recreate some of this. But it might be easier for customized sites to update the fields themselves?

[finnlewis]

@ekes thinks there are a couple more display modes / entity reference fields to upgrade / fix.

To test:

• Install the 2.x branch on Drupal 9.
• Enable the localgov_demo module.
• Run drush updb. This enables the module.
• Then upgrade to Drupal 10.
• The empty module will then get hook_updates to run and can run in Drupal 10.
• Hopefully the geo entites will ahve been migrated to the new geo_entity.

Blocker for LGD 3.x alpha release? Yes!

• we want fresh installs to use this by default.
• the localgov_demo content will need to be updated.
• Stephen: what happens if someone fails to install the 9.x module?
• Ekes: all hook_updates will need to be run.

[ekes]

#77