search

OCA / geospatial

Odoo and GIS
http://oca.github.io/geospatial/index.html
GNU Affero General Public License v3.0
187 stars 281 forks source link

[12.0] fix web view leaflet mal readme #339

Closed legalsylvain closed 1 year ago

legalsylvain commented 1 year ago

/ocabot merge patch

Fastracking.

legalsylvain commented 1 year ago

Annoying.

This trivial PR failed, because of an error in geoengine_base_geolocalize.

2023-05-16 08:06:44,925 252 INFO odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: test_geo_localize (odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner.TestGeoenginePartner) 
2023-05-16 08:06:45,737 252 ERROR odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: FAIL 
2023-05-16 08:06:45,737 252 INFO odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: test_get_geo_point (odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner.TestGeoenginePartner) 
2023-05-16 08:06:45,776 252 INFO odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: ====================================================================== 
2023-05-16 08:06:45,776 252 ERROR odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: FAIL: test_geo_localize (odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner.TestGeoenginePartner) 
2023-05-16 08:06:45,777 252 ERROR odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: Traceback (most recent call last): 
2023-05-16 08:06:45,777 252 ERROR odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: `   File "/__w/geospatial/geospatial/geoengine_base_geolocalize/tests/test_geoengine_partner.py", line 41, in test_geo_localize 
2023-05-16 08:06:45,777 252 ERROR odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: `     "Longitude Should be equals", 
2023-05-16 08:06:45,777 252 ERROR odoo odoo.addons.geoengine_base_geolocalize.tests.test_geoengine_partner: ` AssertionError: 5.41191 != 5.40539 within 2 places : Longitude Should be equals
OCA-git-bot commented 1 year ago

What a great day to merge this nice PR. Let's do it! Prepared branch 12.0-ocabot-merge-pr-339-by-legalsylvain-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-339-by-legalsylvain-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot commented 1 year ago

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-339-by-legalsylvain-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

legalsylvain commented 1 year ago

/ocabot rebase

legalsylvain commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

@legalsylvain The rebase process failed, because command git push --force legalsylvain tmp-pr-339:patch-1 failed with output:

remote: Permission to legalsylvain/geospatial.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/legalsylvain/geospatial/': The requested URL returned error: 403
OCA-git-bot commented 1 year ago

This PR looks fantastic, let's merge it! Prepared branch 12.0-ocabot-merge-pr-339-by-legalsylvain-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-339-by-legalsylvain-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

yvaucher commented 1 year ago

Let me try

/ocabot merge patch

OCA-git-bot commented 1 year ago

This PR looks fantastic, let's merge it! Prepared branch 12.0-ocabot-merge-pr-339-by-yvaucher-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

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