Skip to content

Adjust order quantity based on pack size#11800

Merged
SchrodingersGat merged 3 commits intoinventree:masterfrom
SchrodingersGat:order-pack-size
Apr 26, 2026
Merged

Adjust order quantity based on pack size#11800
SchrodingersGat merged 3 commits intoinventree:masterfrom
SchrodingersGat:order-pack-size

Conversation

@SchrodingersGat
Copy link
Copy Markdown
Member

- Closes inventree#10946

Co-authored-by: Copilot <copilot@github.com>
@SchrodingersGat SchrodingersGat added this to the 1.4.0 milestone Apr 24, 2026
@SchrodingersGat SchrodingersGat added bug Identifies a bug which needs to be addressed User Interface Related to the frontend / User Interface labels Apr 24, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 24, 2026

Deploy Preview for inventree-web-pui-preview canceled.

Name Link
🔨 Latest commit e2f41ef
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/69ed521d64c7b20008cda612

Copy link
Copy Markdown
Member

@matmair matmair left a comment

Choose a reason for hiding this comment

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

Does this depend on #11631 because I do not think quantity_raw is a valid field in the current api

@SchrodingersGat
Copy link
Copy Markdown
Member Author

Does this depend on #11631 because I do not think quantity_raw is a valid field in the current api

This is just a local temporary variable to track the original quantity (without pack size). It is not sent anywhere

@SchrodingersGat SchrodingersGat dismissed matmair’s stale review April 25, 2026 23:46

Variable is just used locally, has no impact on the API

@SchrodingersGat SchrodingersGat merged commit f9cda95 into inventree:master Apr 26, 2026
29 checks passed
@SchrodingersGat SchrodingersGat deleted the order-pack-size branch April 26, 2026 00:23
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.36%. Comparing base (75942fb) to head (e2f41ef).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #11800   +/-   ##
=======================================
  Coverage   91.36%   91.36%           
=======================================
  Files         970      970           
  Lines       50324    50319    -5     
=======================================
- Hits        45977    45976    -1     
+ Misses       4347     4343    -4     
Components Coverage Δ
Backend Apps 91.71% <ø> (+<0.01%) ⬆️
Backend General 93.41% <ø> (ø)
Frontend ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Identifies a bug which needs to be addressed User Interface Related to the frontend / User Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ordering from BO adds item quantity, not pack quantity, to a PO

2 participants