Skip to content

[19.0] [FIX] sale_stock_available_info_popup: write the unit like the other rows - #96

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:19.0-fix-sale_stock_available_info_popup-row
Oct 29, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:19.0-fix-sale_stock_available_info_popup-row

Conversation

@ivantodorovich

@ivantodorovich ivantodorovich commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

The Available to promise row wrote its unit with display_name, unlike the other rows of the popover. It now writes it the same way as they do.

Before:
Screenshot 2026-10-28 at 12 10 35 PM

@OCA-git-bot OCA-git-bot added mod:sale_stock_available_info_popup Module sale_stock_available_info_popup series:19.0 labels Oct 28, 2026
@ivantodorovich
ivantodorovich marked this pull request as ready for review October 28, 2026 15:16
@ivantodorovich
ivantodorovich force-pushed the 19.0-fix-sale_stock_available_info_popup-row branch from 5edddf7 to 6bfb267 Compare October 28, 2026 15:18
@ivantodorovich ivantodorovich changed the title [19.0][FIX] sale_stock_available_info_popup: show the row after the available quantity [19.0] [FIX] sale_stock_available_info_popup: show the row after the available quantity Oct 28, 2026
@ivantodorovich

Copy link
Copy Markdown
Contributor Author

@pedrobaeza

@pedrobaeza pedrobaeza added this to the 19.0 milestone Oct 29, 2026
@pedrobaeza

Copy link
Copy Markdown
Member

Isn't this data interesting to be the first one?

@ivantodorovich

Copy link
Copy Markdown
Contributor Author

The "Available to Promise" is a subset of the "Fresh Available" so it seems more natural to have it below.
I'm ok to revert this if you disagree, my real motivation for this PR is fixing the uom glitch

@pedrobaeza

Copy link
Copy Markdown
Member

Yeah, I think we can still keep it the first, while fixing the UoM thing.

…rows

The Available to promise row showed the unit with display_name, unlike the
other rows of the popover.
@ivantodorovich
ivantodorovich force-pushed the 19.0-fix-sale_stock_available_info_popup-row branch from 6bfb267 to a765ec1 Compare October 29, 2026 14:43
@ivantodorovich ivantodorovich changed the title [19.0] [FIX] sale_stock_available_info_popup: show the row after the available quantity [19.0] [FIX] sale_stock_available_info_popup: write the unit like the other rows Oct 29, 2026
@ivantodorovich

Copy link
Copy Markdown
Contributor Author

It's done!

@pedrobaeza pedrobaeza 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.

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-96-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

It looks like something changed on 19.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 19.0-ocabot-merge-pr-96-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 63f1fa2 into OCA:19.0 Oct 29, 2026
8 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at f6f0272. 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants