Skip to content

Fix stuck-in-past expiry date filter#11518

Merged
SchrodingersGat merged 2 commits intoinventree:masterfrom
jacobfelknor:bug/expiry-filter
Mar 14, 2026
Merged

Fix stuck-in-past expiry date filter#11518
SchrodingersGat merged 2 commits intoinventree:masterfrom
jacobfelknor:bug/expiry-filter

Conversation

@jacobfelknor
Copy link
Copy Markdown
Contributor

I think this may be all that is necessary to fix #11517

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 13, 2026

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

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

@SchrodingersGat
Copy link
Copy Markdown
Member

Nice catch, have you checked if there are any other similar patterns in the code?

@jacobfelknor
Copy link
Copy Markdown
Contributor Author

jacobfelknor commented Mar 13, 2026

I have not, but can

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.06%. Comparing base (6ca102c) to head (726ed82).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11518      +/-   ##
==========================================
- Coverage   88.06%   88.06%   -0.01%     
==========================================
  Files        1296     1296              
  Lines       59074    59082       +8     
  Branches     1935     1935              
==========================================
+ Hits        52025    52032       +7     
- Misses       6568     6569       +1     
  Partials      481      481              
Flag Coverage Δ
backend 89.24% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Backend Apps 91.70% <100.00%> (+<0.01%) ⬆️
Backend General 93.37% <ø> (ø)
Frontend 70.91% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@matmair matmair added bug Identifies a bug which needs to be addressed backport Apply this label to a PR to enable auto-backport action backport-to-1.2.x labels Mar 13, 2026
@matmair matmair added this to the 1.2.4 milestone Mar 13, 2026
@jacobfelknor
Copy link
Copy Markdown
Contributor Author

I see the same pattern in

  1. Build OVERDUE_FILTER
  2. Order line item OVERDUE_FILTER

I'll fix

@SchrodingersGat SchrodingersGat added the api Relates to the API label Mar 14, 2026
@SchrodingersGat
Copy link
Copy Markdown
Member

@jacobfelknor thanks for digging into this one and providing a fix :)

@SchrodingersGat SchrodingersGat merged commit c3cedd2 into inventree:master Mar 14, 2026
38 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
1.2.x Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

backport --pr 11518

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

SchrodingersGat pushed a commit to SchrodingersGat/InvenTree that referenced this pull request Mar 14, 2026
* fix stuck-in-past expiry date filter

* also fix OVERDUE_FILTER date problem
SchrodingersGat added a commit that referenced this pull request Mar 15, 2026
* fix stuck-in-past expiry date filter

* also fix OVERDUE_FILTER date problem

Co-authored-by: Jacob Felknor <jacobfelknor073@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api Relates to the API backport Apply this label to a PR to enable auto-backport action bug Identifies a bug which needs to be addressed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expired Filter Check Stuck at Server Start Date

3 participants