Skip to content

[19.0][FIX] sale_order_line_description: description update#4437

Open
chienandalu wants to merge 1 commit into
OCA:19.0from
moduon:19.0-fix-sale_order_line_description-description-update
Open

[19.0][FIX] sale_order_line_description: description update#4437
chienandalu wants to merge 1 commit into
OCA:19.0from
moduon:19.0-fix-sale_order_line_description-description-update

Conversation

@chienandalu

@chienandalu chienandalu commented Jul 1, 2026

Copy link
Copy Markdown
Member

When we changed the description manually, the product name was readded by the SaleOrderLineProductField js mechanisms.

  • Backend tests improved
  • Added tour as regression test

https://www.loom.com/share/f73a409780874fc7b34db550499707bb

cc @moduon MT-15180

please review @yajo @Gelojr

@OCA-git-bot OCA-git-bot added series:19.0 mod:sale_order_line_description Module sale_order_line_description labels Jul 1, 2026

@yajo yajo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code review.

@chienandalu
chienandalu force-pushed the 19.0-fix-sale_order_line_description-description-update branch from 4c1fa4b to b88f40d Compare July 1, 2026 09:37
When we changed the description manually, the product name was readded by the
SaleOrderLineProductField js mechanisms.

MT-15180
@chienandalu
chienandalu force-pushed the 19.0-fix-sale_order_line_description-description-update branch from b88f40d to 9cd272a Compare July 1, 2026 10:10

@BhaveshHeliconia BhaveshHeliconia left a comment

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.

LGTM!

@yajo

yajo commented Jul 17, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-4437-by-yajo-bump-patch, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

@yajo your merge command was aborted due to failed check(s), which you can inspect on this commit of 19.0-ocabot-merge-pr-4437-by-yajo-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

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.

5 participants