[17.0] [MIG] edi_storage_oca: Migration to 17.0 #114
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
luisDIXMIT
force-pushed
the
17.0-mig-edi_storage_oca
branch
from
f4e3cfb to
2cd048b
Compare
etobella
approved these changes
Dec 19, 2024
etobella
force-pushed
the
17.0-mig-edi_storage_oca
branch
2 times, most recently
from
29f5d89 to
31386e2
Compare
edi_storage_oca/__manifest__.py
Outdated
| @@ -7,7 +7,7 @@ | |||
| "summary": """ | |||
| Base module to allow exchanging files via storage backend (eg: SFTP). | |||
| """, | |||
| "version": "16.0.1.1.2", | |||
| "version": "17.0.1.1.2", | |||
There was a problem hiding this comment.
@luisDIXMIT could you reset the version numbering to 17.0.1.0.0 ?
etobella
approved these changes
Jan 20, 2025
|
@luisDIXMIT can you remove my commit and add @JordiMForgeFlow commit from the PR? |
Ack files should be treated as incoming records. Receive files is not handled yet by the framework and will be added soon.
* A cron job check related storage on edi backends to create the needed exchange files. * Add basic receive component.
…ge.record to avoid moving files after processing
Enable any internal user to trigger creation of EDI records without EDI Manager group nor read access to fs.storage
When relying on listing all files in a dir, if there's a missing trailing slash no file will be found.
Currently translated at 100.0% (19 of 19 strings) Translation: edi-framework-16.0/edi-framework-16.0-edi_storage_oca Translate-URL: https://translation.odoo-community.org/projects/edi-framework-16-0/edi-framework-16-0-edi_storage_oca/it/
Currently translated at 100.0% (19 of 19 strings) Translation: edi-framework-16.0/edi-framework-16.0-edi_storage_oca Translate-URL: https://translation.odoo-community.org/projects/edi-framework-16-0/edi-framework-16-0-edi_storage_oca/it/
luisDIXMIT
force-pushed
the
17.0-mig-edi_storage_oca
branch
from
31386e2 to
9ea0740
Compare
Done |
It was controlled later by fs.exists, but this code was incompatible with some fsspecs.
luisDIXMIT
force-pushed
the
17.0-mig-edi_storage_oca
branch
from
9ea0740 to
948543e
Compare
JordiMForgeFlow
approved these changes
Jan 20, 2025
|
This PR has the |
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at a2428b6. 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Standard migration.
The tests are failing due to an error in the code of the edi_oca module, which was fixed in PR #113.
@etobella