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

Relation update #8524

Merged
merged 7 commits into from
Nov 19, 2024
Merged

Conversation

SchrodingersGat
Copy link
Member

@SchrodingersGat SchrodingersGat added part Related to Part models api Relates to the API Platform UI Related to the React based User Interface labels Nov 19, 2024
@SchrodingersGat SchrodingersGat added this to the 0.17.0 milestone Nov 19, 2024
Copy link

netlify bot commented Nov 19, 2024

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

Name Link
🔨 Latest commit afb0f2f
🔍 Latest deploy log https://app.netlify.com/sites/inventree-web-pui-preview/deploys/673cf9f9e470220009760275
😎 Deploy Preview https://deploy-preview-8524--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: 100 (🟢 up 1 from production)
Accessibility: 86 (no change from production)
Best Practices: 100 (no change from production)
SEO: 78 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

Copy link

codecov bot commented Nov 19, 2024

Codecov Report

Attention: Patch coverage is 77.77778% with 6 lines in your changes missing coverage. Please review.

Project coverage is 84.63%. Comparing base (2bc6ddb) to head (afb0f2f).
Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
src/backend/InvenTree/part/api.py 76.47% 4 Missing ⚠️
src/frontend/src/tables/part/RelatedPartTable.tsx 33.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8524      +/-   ##
==========================================
+ Coverage   84.52%   84.63%   +0.10%     
==========================================
  Files        1176     1178       +2     
  Lines       53555    53580      +25     
  Branches     2023     2025       +2     
==========================================
+ Hits        45265    45345      +80     
+ Misses       7785     7709      -76     
- Partials      505      526      +21     
Flag Coverage Δ
backend 85.96% <83.33%> (+0.01%) ⬆️
pui 69.36% <33.33%> (+0.57%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@wolflu05
Copy link
Contributor

We might could also overengineer this by adding a second table like relation types which can be selected. That way we can keep this field much clear/filter/… by common relations types. E.g. show all additional equipment relations, … But that’s just a thought

@matmair
Copy link
Member

matmair commented Nov 19, 2024

Using a state would probably be user-friendly as custom states would make it easy to extend the selection without adding new interfaces that need management functionality and discovery.
That could be added as a separate PR to keep this one in the imminent 0.17.0

@SchrodingersGat SchrodingersGat merged commit 7fcf068 into inventree:master Nov 19, 2024
26 checks passed
@SchrodingersGat SchrodingersGat deleted the relation-update branch November 19, 2024 21:54
@wolflu05
Copy link
Contributor

Using a state would probably be user-friendly as custom states would make it easy to extend the selection without adding new interfaces that need management functionality and discovery. That could be added as a separate PR to keep this one in the imminent 0.17.0

@matmair should we create an issue to track this then? Maybe you could do that with you ideas about the custom states? I'm not so familiar with the new state system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Relates to the API part Related to Part models Platform UI Related to the React based User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants