[FIX] remove pin version of dateparser

[florian-dacosta]

Issue reported here #544 is fixed in recent version of dateparser.
Fixed here I believe : scrapinghub/dateparser#1045

[OCA-git-bot]

Hi @alexis-via,
some modules you are maintaining are being modified, check this out!

[florian-dacosta]

@simahawk

[simahawk]

shall we pin dateparser >= $minor_version_with_the_fix?

[bosd]

Please fasttrack this. As the module is not installable due to a version conflict!

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[bosd]

shall we pin dateparser >= $minor_version_with_the_fix

LGTM

[rvalyi]

/ocabot merge patch

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-716-by-rvalyi-bump-patch, awaiting test results.

[OCA-git-bot]

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

[bosd]

@florian-dacosta @rvalyi Can you attend to this?

[rvalyi]

/ocabot merge patch

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-716-by-rvalyi-bump-patch, awaiting test results.

[OCA-git-bot]

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

[bosd]

Why does it fail?

[rvalyi]

@bosd @florian-dacosta the merge failed because dateparser was still pinned to the old version in edi_pdf2data_oca/__manifest__.py so the requirements.txt generation would still pick the pinned version and be different than it was in the PR.
I fixed that in a fixup! commit.

Still OK with the PR?

[bosd]

Yes, please proceed

[OCA-git-bot]

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). 🤖

[rvalyi]

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-716-by-rvalyi-bump-patch, awaiting test results.

[OCA-git-bot]

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