Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15.0][IMP] stock_barcodes: Add update_barcodes_wiz_after_changes method #662

Open
wants to merge 4 commits into
base: 15.0
Choose a base branch
from

Conversation

carlosdauden
Copy link
Contributor

@Tecnativa TT52013

@carlosdauden carlosdauden force-pushed the 15.0-IMP-stock_barcodes-add-update_barcodes_wiz_after_changes branch from 961dc7d to 93c33f3 Compare December 4, 2024 16:42
@carlosdauden carlosdauden force-pushed the 15.0-IMP-stock_barcodes-add-update_barcodes_wiz_after_changes branch from ec6a532 to 413b1e1 Compare December 5, 2024 14:09
CarlosRoca13 and others added 2 commits December 10, 2024 12:47
…de_scanned instead of onchange

Onchange limits documented in method:
It is not possible for a ``one2many`` or ``many2many`` field to modify
itself via onchange. This is a webclient limitation - see `#2693 <https://github.com/odoo/odoo/issues/2693>`_.

This causes refresh problems when creating or modifying records, and it is also not ideal to create records in "onchange mode"

Backported from OCA@725221c#diff-dcdad8cf227ac6f9016ad537c1aca9cacab5576bfd5e246507cc5e096a238008L497
@sergio-teruel sergio-teruel force-pushed the 15.0-IMP-stock_barcodes-add-update_barcodes_wiz_after_changes branch from 1d04efd to fe2f4c4 Compare January 8, 2025 11:09
@sergio-teruel
Copy link
Contributor

@carlosdauden set PR ready to review

@carlosdauden carlosdauden marked this pull request as ready for review January 8, 2025 13:17
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