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

[16.0][MIG] pos_lot_selection: Migration to 16.0 #917

Closed
wants to merge 14 commits into from
Closed

[16.0][MIG] pos_lot_selection: Migration to 16.0 #917

wants to merge 14 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jan 16, 2023

No description provided.

chienandalu and others added 14 commits January 16, 2023 09:12
With PR odoo/odoo#23698 merged, clone control is
no longer needed for lots.

[FIX] pos_lot_selection: group lots by quants

fixup! [FIX] pos_lot_selection: group lots by quants

fixup! fixup! [FIX] pos_lot_selection: group lots by quants
Object.assign is introduced in ES6 (2015) which is not supported by
PhantomJS. Replace it with the equivalent _.extend function to prevent
frontend tests from failing.

1. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign#Specifications
2. ariya/phantomjs#14506
Currently translated at 100.0% (1 of 1 strings)

Translation: pos-11.0/pos-11.0-pos_lot_selection
Translate-URL: https://translation.odoo-community.org/projects/pos-11-0/pos-11-0-pos_lot_selection/ca/
Regular Point of Sale Users with the minimal access rights are not
allowed to read the `stock.production.lot` model.
@ivantodorovich
Copy link
Contributor

CI is red :(

@ivantodorovich ivantodorovich added this to the 16.0 milestone Feb 15, 2023
@ivantodorovich
Copy link
Contributor

/ocabot migration pos_lot_selection

@OCA-git-bot OCA-git-bot mentioned this pull request Feb 15, 2023
37 tasks
Copy link

@ajaniszewska-dev ajaniszewska-dev left a comment

Choose a reason for hiding this comment

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

seems good

//edit: can you please check test?

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.

8 participants