Skip to content

[6417][IMP] stock_view_adj: add stock.piking link in stock.move_tree_view#141

Open
nobuQuartile wants to merge 2 commits into16.0from
6417-imp-stock_view_adj
Open

[6417][IMP] stock_view_adj: add stock.piking link in stock.move_tree_view#141
nobuQuartile wants to merge 2 commits into16.0from
6417-imp-stock_view_adj

Conversation

@nobuQuartile
Copy link
Copy Markdown
Contributor

@nobuQuartile nobuQuartile force-pushed the 6417-imp-stock_view_adj branch from ae86c62 to 5076871 Compare March 10, 2026 02:32
@nobuQuartile nobuQuartile changed the title [6417][IMP] stock_view_adj [6417][IMP] stock_view_adj add stock.piking link in stock.move_tree_view Mar 10, 2026
@nobuQuartile nobuQuartile changed the title [6417][IMP] stock_view_adj add stock.piking link in stock.move_tree_view [6417][IMP] stock_view_adj/ add stock.piking link in stock.move_tree_view Mar 10, 2026
@nobuQuartile nobuQuartile force-pushed the 6417-imp-stock_view_adj branch from 5076871 to 503ee59 Compare March 10, 2026 02:33
@nobuQuartile nobuQuartile changed the title [6417][IMP] stock_view_adj/ add stock.piking link in stock.move_tree_view [6417][IMP] stock_view_adj: add stock.piking link in stock.move_tree_view Mar 10, 2026
Copy link
Copy Markdown
Contributor

@AungKoKoLin1997 AungKoKoLin1997 left a comment

Choose a reason for hiding this comment

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

Code Review: LGTM
Just minor point.

<field name="inherit_id" ref="stock.view_move_form" />
<field name="arch" type="xml">
<xpath expr="//field[@name='reference']" position="after">
<field name="picking_id" readonly="1" />
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think readonly attribute is needed because that form has create="0" edit="0" attributes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants