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.

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

[14.0] edi_oca: backports from v16 #1052

Merged
merged 17 commits into from
Nov 15, 2024

Conversation

simahawk
Copy link
Contributor

@simahawk simahawk commented Oct 11, 2024

Misc commits from OCA/edi-framework

@OCA-git-bot
Copy link
Contributor

Hi @etobella,
some modules you are maintaining are being modified, check this out!

@simahawk simahawk force-pushed the oca-port-edi_oca-16.0-to-14.0-4e10d8 branch from 5ca2f6d to 00b6c42 Compare October 25, 2024 08:06
@simahawk simahawk marked this pull request as ready for review October 25, 2024 08:29
@simahawk
Copy link
Contributor Author

simahawk commented Oct 25, 2024

@sebalix Reminder for us: oca-port blacklisting does not consider the repo. The PRs listed in the blacklist are from edi-framework. OCA/oca-port#54

@simahawk
Copy link
Contributor Author

simahawk commented Nov 11, 2024

@sebalix Reminder for us: oca-port blacklisting does not consider the repo. The PRs listed in the blacklist are from edi-framework. OCA/oca-port#54

The issue has been fixed. TODO: update blacklisting here. ✔️

simahawk and others added 17 commits November 15, 2024 15:34
Select main  only as they can be nested into field custom forms.
I'm looking at you  on v16 :S
this field can be used in backend views to show/hide configuration elements depending on the backend type
Currently when we receive an empty file:

- Receive step doesn't end up in state error_on_receive : that's because we just ignore the content here
- But the process step will raise an error
- So this commit goal tries to improve that
- By default, this option is disabled so we still consider that an empty file is an error case
@simahawk simahawk force-pushed the oca-port-edi_oca-16.0-to-14.0-4e10d8 branch from 00b6c42 to 9155585 Compare November 15, 2024 14:34
@OCA-git-bot
Copy link
Contributor

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

@simahawk
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-1052-by-simahawk-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 678319d into OCA:14.0 Nov 15, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 0a85e71. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.