Skip to content

Conversation

@AlexPForgeFlow
Copy link
Contributor

Migration to 17.0.

  • EDI Configuration group moved to EDI page from edi_oca

<page name="extra" position="after">
<page name="edi" string="EDI">
<group name="edi_stock_group" string="Electronic Data Interchange">
<field name="edi_disable_auto" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be readonly if picking is done or cancelled

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch! 👀

Copy link

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM!

@AlexPForgeFlow AlexPForgeFlow force-pushed the 17.0-mig-edi_stock_oca branch from 388089b to 761527c Compare May 12, 2025 08:59
@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). 🤖

@john-herholz-dt
Copy link
Contributor

@simahawk can you merge this PR?

@github-actions
Copy link

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.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Nov 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved ready to merge stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants