search

OCA / search-engine

GNU Affero General Public License v3.0
45 stars 60 forks source link

[16.0][FIX] search_engine_image_thumbnail: Fix size lookup #174

Closed lmignon closed 5 months ago

lmignon commented 9 months ago

cc @sebastienbeau @qgroulard @marielejeune @sbejaoui

lmignon commented 9 months ago

/ocabot merge patch

OCA-git-bot commented 9 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-174-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot commented 9 months ago

@lmignon The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name search-engine --branch 16.0 --addons-dir /tmp/tmp58hyeugp --commit failed with output:

Both .md and .rst fragments found in /tmp/tmp58hyeugp/search_engine_image_thumbnail/readme. Please ensure the same format is used for all fragments.
lmignon commented 6 months ago

/ocabot merge patch

OCA-git-bot commented 6 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-174-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot commented 6 months ago

@lmignon The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name search-engine --branch 16.0 --addons-dir /tmp/tmpnr12q4ai --commit failed with output:

Both .md and .rst fragments found in /tmp/tmpnr12q4ai/search_engine_image_thumbnail/readme. Please ensure the same format is used for all fragments.
lmignon commented 5 months ago

/ocabot merge patch

OCA-git-bot commented 5 months ago

This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-174-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot commented 5 months ago

@lmignon The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name search-engine --branch 16.0 --addons-dir /tmp/tmp8hqqppt6 --commit failed with output:

Both .md and .rst fragments found in /tmp/tmp8hqqppt6/search_engine_image_thumbnail/readme. Please ensure the same format is used for all fragments.
sbidoul commented 5 months ago

@lmignon can you retry?

lmignon commented 5 months ago

/ocabot merge patch

OCA-git-bot commented 5 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-174-by-lmignon-bump-patch, awaiting test results.

OCA-git-bot commented 5 months ago

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

lmignon commented 5 months ago

@lmignon can you retry?

It works. Thank you.