Skip to content

[18.0][IMP] edi_storage_oca: post-process input file via conf#280

Open
ArnauCForgeFlow wants to merge 1 commit into
OCA:18.0from
ForgeFlow:18.0-imp-edi_storage_oca-edi_configuration_move
Open

[18.0][IMP] edi_storage_oca: post-process input file via conf#280
ArnauCForgeFlow wants to merge 1 commit into
OCA:18.0from
ForgeFlow:18.0-imp-edi_storage_oca-edi_configuration_move

Conversation

@ArnauCForgeFlow
Copy link
Copy Markdown
Contributor

@ArnauCForgeFlow ArnauCForgeFlow commented May 19, 2026

Reintroduce the event-driven file move "on_edi_exchange_done" / "on_edi_exchange_error" that was lost when the "edi_component_oca" dependency was dropped, using the global edi.configuration mechanism added to "edi_core_oca" in #275:

  • Add _move_file on fs.storage: looks up the file in the source directory and schedules an fs.move via a post-commit hook.
  • Add _storage_on_edi_exchange_done / _storage_on_edi_exchange_error on edi.backend, mirroring the original listener behaviour.
  • Ship two default global edi.configuration records bound to the matching triggers. They are inactive by default and require an explicit backend_id before being enabled.

Depends on:

@simahawk

Comment thread edi_storage_oca/data/edi_configuration.xml Outdated
Comment thread edi_storage_oca/models/fs_storage.py Outdated
Comment thread edi_storage_oca/static/description/index.html
Comment thread edi_storage_oca/tests/test_event_listener.py
@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-imp-edi_storage_oca-edi_configuration_move branch from 627cd32 to ae3d52b Compare May 20, 2026 07:04
@ArnauCForgeFlow
Copy link
Copy Markdown
Contributor Author

Rebased!

@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-imp-edi_storage_oca-edi_configuration_move branch from ae3d52b to aa224b9 Compare May 20, 2026 07:50
@ArnauCForgeFlow ArnauCForgeFlow force-pushed the 18.0-imp-edi_storage_oca-edi_configuration_move branch from aa224b9 to 295b1fa Compare May 20, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants