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.

Sign up for GitHub

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

Jump to bottom

[17.0][MIG] base_geoengine_demo: Migration to 17.0 #372

Merged
merged 93 commits into from
Oct 17, 2024
Merged

[17.0][MIG] base_geoengine_demo: Migration to 17.0 #372

merged 93 commits into from
Oct 17, 2024

Conversation

peluko00
Copy link

@peluko00 peluko00 commented May 21, 2024 •
edited
Loading

@peluko00 peluko00 force-pushed the 17.0-mig-base_geoengine_demo branch 2 times, most recently from 0f95f19 to 7ae2c6a Compare May 21, 2024 11:51
miquelalzanillas
miquelalzanillas approved these changes May 21, 2024
View reviewed changes
Copy link

@miquelalzanillas miquelalzanillas left a comment

Choose a reason for hiding this comment

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

Tested functionally. LGTM.

@miquelalzanillas miquelalzanillas mentioned this pull request May 21, 2024
Merged
3 tasks
@miquelalzanillas
Copy link

miquelalzanillas commented May 22, 2024 •
edited
Loading

Tested functionally. LGTM.

Sorry but I didn't tested following the 'base_geoengine_demo' readme and trying to follow those indications still are some bugs.

@peluko00 peluko00 force-pushed the 17.0-mig-base_geoengine_demo branch from 7ae2c6a to 6d5eaa4 Compare June 13, 2024 07:18
@peluko00 peluko00 force-pushed the 17.0-mig-base_geoengine_demo branch 2 times, most recently from 547efc5 to 2c895f1 Compare September 27, 2024 12:25
@peluko00
Copy link
Author

It's ready to be merge @dreispt

@dreispt
Copy link
Member

dreispt commented Sep 27, 2024

@peluko00 Can you please rebase on latest 17.0, and ensure that that last Merge commit is not in the history?

nbessi and others added 20 commits September 30, 2024 08:22
@nbessi @peluko00
[ADD] Demo module \o/
fcfb6f6
@nbessi @peluko00
[FIX] fields_view_get that broke common behavior of function
b9cdd5c
@nbessi @peluko00
[ADD] search to retail machine demo
26c6c10
@nbessi @peluko00
[FIX] fields_view_get support map view without id
5e519af
@nbessi @peluko00
[DATA] creating more complete set of demo views
109910b
@nbessi @peluko00
[add total sales in zip view]
6f50275
@nbessi @peluko00
[FIX] compatiblity with new BaseModel class
dda79eb
@nbessi @peluko00
[IMP] Demo data
0a5fabf 
+ [FIX] in demo in order to have multipolygon samples
@nbessi @peluko00
[ADD] support of spatial operator in geo_engine
1511e30
@nbessi @peluko00
[ADD] geometry operator sample
1d598c3 
+ [IMP] menu name
@nbessi @peluko00
[ADD] geoname bulk wizzard + small fixes
89b77a5
@nbessi @peluko00
[IMP] demo is now self-contained + fix in __openerp__.py
48f4a77
@nbessi @peluko00
[IMP] __openerp__ icon and coding
b348bd7 
+ [ADD] security
@nbessi @peluko00
[FIX] import to be compatible with python 2.6 amnd 2.7
9f6f992
@peluko00
[FIX] license in base_geoengine_demo
291099d
@nbessi @peluko00
[IMP] use edit map widget in geoengine base demo
78503ed
@nbessi @peluko00
[IMP] adapt base demo to new edit widget
7f60d2c
@nbessi @peluko00
[IMP] demo data now use map box
924e440 
+ replace icons by nicer icons
@nbessi @peluko00
[INIT] V7.0 serie
039c219
@peluko00
Using geoengine view properly
84b9780
@peluko00 peluko00 force-pushed the 17.0-mig-base_geoengine_demo branch 2 times, most recently from 1712894 to 3223b4c Compare September 30, 2024 06:31
@peluko00 peluko00 force-pushed the 17.0-mig-base_geoengine_demo branch from 3223b4c to e9f0b76 Compare September 30, 2024 06:31
@peluko00
Copy link
Author

@peluko00 Can you please rebase on latest 17.0, and ensure that that last Merge commit is not in the history?

Done, thanks! Now it's ready

@yvaucher
Copy link
Member

/ocabot migration base_geoengine_demo

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Oct 17, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 17, 2024
Open
4 tasks
@peluko00
Copy link
Author

It's ready to merge @yvaucher

@yvaucher
Copy link
Member

Indeed I saw the red mark but it's only the coverage. Thanks .

@yvaucher
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-372-by-yvaucher-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a28b280 into OCA:17.0 Oct 17, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at aed2440. Thanks a lot for contributing to OCA. ❤️

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

@miquelalzanillas miquelalzanillas miquelalzanillas approved these changes

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
17.0
Development

Successfully merging this pull request may close these issues.