search

OCA / edi-framework

GNU Affero General Public License v3.0
6 stars 26 forks source link

[16.0] edi_storage: handle missing trailing slash #81

Closed simahawk closed 2 months ago

simahawk commented 3 months ago

When relying on listing all files in a dir, if there's a missing trailing slash no file will be found.

OCA-git-bot commented 3 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

dreispt commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-81-by-dreispt-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-81-by-dreispt-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 3 months ago

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-81-by-dreispt-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.

simahawk commented 3 months ago

/ocabot merge patch

OCA-git-bot commented 3 months ago

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

OCA-git-bot commented 2 months ago

@simahawk your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-81-by-simahawk-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.

dreispt commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

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

OCA-git-bot commented 2 months ago

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