Skip to content

[Feature] Non-Conformance - #12682

Draft
SchrodingersGat wants to merge 23 commits into
inventree:masterfrom
SchrodingersGat:ncr
Draft

[Feature] Non-Conformance#12682
SchrodingersGat wants to merge 23 commits into
inventree:masterfrom
SchrodingersGat:ncr

Conversation

@SchrodingersGat

Copy link
Copy Markdown
Member

@SchrodingersGat SchrodingersGat added this to the 1.6.0 milestone Aug 22, 2026
@SchrodingersGat SchrodingersGat added api Relates to the API feature migration Data or schema migrations User Interface Related to the frontend / User Interface full-run Always do a full QC CI run labels Aug 22, 2026
@netlify

netlify Bot commented Aug 22, 2026

Copy link
Copy Markdown

Deploy Preview for inventree-web-pui-preview ready!

Name Link
🔨 Latest commit 8d9e9f2
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/6a937bfc469c7e0008f14682
😎 Deploy Preview https://deploy-preview-12682--inventree-web-pui-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 93 (🟢 up 1 from production)
Accessibility: 82 (no change from production)
Best Practices: 92 (no change from production)
SEO: 78 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@SchrodingersGat
SchrodingersGat marked this pull request as draft August 22, 2026 04:01
@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.70867% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.89%. Comparing base (a205717) to head (8d9e9f2).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12682      +/-   ##
==========================================
- Coverage   86.92%   86.89%   -0.04%     
==========================================
  Files        1462     1464       +2     
  Lines       98187    98627     +440     
  Branches    11479    11232     -247     
==========================================
+ Hits        85352    85701     +349     
- Misses      12771    12860      +89     
- Partials       64       66       +2     
Flag Coverage Δ
backend 90.94% <97.70%> (+<0.01%) ⬆️
migrations 38.51% <45.00%> (+0.07%) ⬆️

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

Components Coverage Δ
Backend Apps 92.19% <97.70%> (-0.02%) ⬇️
Backend General 93.19% <ø> (-0.60%) ⬇️
Frontend 79.69% <ø> (-0.18%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@matmair

matmair commented Aug 22, 2026

Copy link
Copy Markdown
Member

Very interesting for bettering MES area of the platform.
As this is an advanced concept that might not be enabled on most sites, I think this would be a great candidate for a built-in plugin as there is not much interaction with the rest of the platform. This could be used to detect and discuss implementation patterns that are non ergonomic (the way FSM transitions are exposed through APIs for example).

Comment on lines +1133 to +1147
class NCRMixin:
"""Mixin class for NonConformance (NCR) API endpoints."""

queryset = NonConformance.objects.all()
serializer_class = build.serializers.NonConformanceSerializer


class NCRListOutputOptions(OutputConfiguration):
"""Output options for the NCRList endpoint."""

OPTIONS = [InvenTreeOutputOption('part_detail', default=True)]


class NCRList(NCRMixin, OutputOptionsMixin, ListCreateAPI):
"""API endpoint for accessing a list of NonConformance (NCR) objects.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

implementing this via a viewset might be more efficient (see #12317)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

see also #12693

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

Labels

api Relates to the API feature full-run Always do a full QC CI run migration Data or schema migrations User Interface Related to the frontend / User Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Build Order Non Conformance

2 participants