Skip to content

[18.0][MIG] base_geoengine_demo - #483

Open
weinni2000 wants to merge 102 commits into
OCA:18.0from
weinni2000:18.0-mig-base_geoengine_demo
Open

weinni2000 wants to merge 102 commits into
OCA:18.0from
weinni2000:18.0-mig-base_geoengine_demo

Conversation

@weinni2000

@weinni2000 weinni2000 commented Sep 29, 2026 •

Copy link
Copy Markdown
  • _check_the_point: loop over the records instead of reading self.the_point / self.zip_id (failed on multi-record writes).
  • _compute_zip_id: clear zip_id with False instead of 0, which gave the nonexistent record dummy.zip(0,) when the point was removed.
  • Add tests for the zip computation, the total sales and the point/zip constraint.

Migrated with odev.

Assisted-by: Claude Code (claude-opus-5-5)

nbessi and others added 30 commits September 29, 2026 09:45
+ [FIX] in demo in order to have multipolygon samples
+ replace icons by nicer icons
geoengine_raster_layer: mapbox_type -> mapbox_id
Merged some code from 6.1 branch
The group element is required to properly display labels on the form

+ list all know contributors from git
This is a workaround of issue odoo/odoo#5618
Where non stored fields are not set into ir.models.fields table
As long wa do not have implemented a geodepends decorator
Store values will never be refreshed
@weinni2000
weinni2000 force-pushed the 18.0-mig-base_geoengine_demo branch from 0bf24c2 to 3ee9f52 Compare November 1, 2026 09:47
@oca-cla-bot

oca-cla-bot Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

The following commit authors need to sign the Contributor License Agreement:

CLA not signed

@weinni2000

Copy link
Copy Markdown
Author

Hi @Tonow-c2c @sersanchus @yvaucher @kouffsamuel, your earlier commits are part of this migration, and the CLA check fails for thomas.nowicki@camptocamp.com, sersanchus@gmail.com, yannick.vaucher@camptocamp.com and samuel.kouff@hotmail.com. Could you sign at https://clabot.odoo-community.org/ (or have them added to your company's CLA)? Thanks!

@yvaucher

yvaucher commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

The 2 employee at savoir-faire-linux are long gone.

@Tonow-c2c is probably an inactive account as he is not part of C2C anymore. His CLA was signed at the time.

The nbessi email seems odd it was a misconfiguration of Nicolas Bessi account at the time. Should have been nicolas.bessi@camptocamp.com at the time (that email is invalid now)

As for my old email the CLA was signed. That email is not valid anymore. Even if I could I don't want to add back an invalid email to my account that I couldn't validate nor control.

This is due to the loss of history of signed CLA

@weinni2000

Copy link
Copy Markdown
Author

@Tonow-c2c is probably an inactive account as he is not part of C2C anymore. His CLA was signed at the time.

As for my old email the CLA was signed. That email is not valid anymore. Even if I could I don't want to add back an invalid email to my account that I couldn't validate nor control.

This is due to the loss of history of signed CLA

anybody knows who is responsible in the OCA for this new feature? and how to sort out old contributors?

@yvaucher

yvaucher commented Nov 3, 2026

Copy link
Copy Markdown
Member

On the mailing list @sbidoul mentionned that he will attempt to scrap some data. I hope we have at least a copy of the old CLAs.

Hi everyone,

We did import the old data in the new bot database, of course.

The problem is for people whose email changed over time, as we migrated only the currently known email for each ICLA and ECLA.

I plan to check if migrating more historical data is possible, but it's not yet clear if that can be done.

Best regards,

-Stéphane

@weinni2000

Copy link
Copy Markdown
Author

@etobella could you add CLA exceptions for these addresses? They are authors of earlier commits that this migration keeps, and they can't sign because the mail bounces:

Like the exceptions you added for OCA/account-invoicing#2507. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:base_geoengine_demo Module base_geoengine_demo series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.