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

🎉1️⃣0️⃣ pos_inventory_adjustment #1285

Draft
wants to merge 3 commits into
base: 10.0
Choose a base branch
from

Conversation

});
if (inv_order) {
self.set_order(inv_order);
} else {
self.get_inventory_stages([inv_id[0]]).then(function(inv) {
inv = inv[0];
var lines = self
.get_inventory_stage_lines(inv.line_ids)
self.get_inventory_stage_lines(inv.line_ids)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yelizariev Я же ничего этим не ломаю? JS'ка иногда ломает мне голову

Copy link
Collaborator

Choose a reason for hiding this comment

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

Вроде нормально -- lines нигде не используется

@trojikman trojikman force-pushed the 10.0-pos_inventory_adjustment branch from 0468b89 to e0ed143 Compare July 1, 2020 19:57
@trojikman trojikman force-pushed the 10.0-pos_inventory_adjustment branch from e0ed143 to 327f1c2 Compare July 2, 2020 21:49
"license": "Other OSI approved licence", # MIT
"price": 29.00,
"currency": "EUR",
"depends": ["point_of_sale", "stock", "staged_inventory"],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

выяснилось, что staged_inventory это сторонняя зависимость

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

@yelizariev yelizariev Jul 3, 2020

Choose a reason for hiding this comment

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

Надо автора спросить поменять лицензию и тогда сможем в misc-addons опубликовать. Иначе всё отменяется

@trojikman trojikman marked this pull request as draft May 28, 2021 10:45
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.

None yet

2 participants