Skip to content

[16.0][FIX] pos_order_reorder: Conditional Display of "Repeat Order" Button Based on POS Setting#1418

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
factorlibre:16.0-fix-pos_order_reorder
Jul 23, 2025
Merged

[16.0][FIX] pos_order_reorder: Conditional Display of "Repeat Order" Button Based on POS Setting#1418
OCA-git-bot merged 1 commit into
OCA:16.0from
factorlibre:16.0-fix-pos_order_reorder

Conversation

@SuShanVoong

Copy link
Copy Markdown

Description:

The "Allow Reorder" setting in the POS configuration was not functioning correctly. Regardless of whether the option was enabled or disabled, the "Repeat Order" button remained visible in the interface, potentially causing confusion and unintended duplicate orders.

Cases Covered with This Improvement:

  • Prevents the "Re-orderr" button from being displayed when the option is disabled in the POS settings.

  • Ensures the feature behaves according to the user-defined configuration.

  • Reduces the risk of incorrect order repetition in the sales environment.

Implemented Solution:

A condition has been added to the code to control the visibility of the "Re-order" button based on the current value of the "Allow Reorder" setting. This ensures that the button is only shown when the option is enabled, aligning the functionality with the configured behavior in the POS.

FL-571-6323

@SuShanVoong

Copy link
Copy Markdown
Author

Development Testing:
Grabación de pantalla desde 2025-07-23 15-09-50.webm

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

This was fixed in migrations to newer versions, such as 17.0 in #1304, but not ported to 16.0. Thanks for solving it!

Conditional Display of "Repeat Order" Button Based on POS Setting
@SuShanVoong SuShanVoong force-pushed the 16.0-fix-pos_order_reorder branch from eb9edde to ce0ed68 Compare July 23, 2025 13:40
@legalsylvain

Copy link
Copy Markdown
Contributor

Thanks.

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1418-by-legalsylvain-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5a85909 into OCA:16.0 Jul 23, 2025
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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