Skip to content

fix: stocks always including values for all materials#994

Merged
tom-rm-meyer-ISST merged 2 commits intoeclipse-tractusx:mainfrom
achtzig20:fix/stock-distinction-by-materialnumber
Oct 1, 2025
Merged

fix: stocks always including values for all materials#994
tom-rm-meyer-ISST merged 2 commits intoeclipse-tractusx:mainfrom
achtzig20:fix/stock-distinction-by-materialnumber

Conversation

@ReneSchroederLJ
Copy link
Copy Markdown
Member

@ReneSchroederLJ ReneSchroederLJ commented Oct 1, 2025

Description

  • added ownMaterialNumber parameter to stockView/material-stocks and stockView/product-stocks
  • added ownMaterialNumber parameter to useStocks hook
  • adjusted bruno requests to accommodate the change

resolves #979

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

  • DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
  • Copyright and license header are present on all affected files
  • If helm chart has been changed, the chart version has been bumped to either next major, minor or patch level (compared to released chart).

Copy link
Copy Markdown
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

Thanks for your quick response! Works great!

As the required parameter is a breaking change, we need to bump the version as a major - app and chart version.

I've sent you a message with a proposal for the admin guide (take care of the links).

Comment thread backend/pom.xml Outdated
Comment thread charts/puris/Chart.yaml Outdated
Comment thread frontend/package.json Outdated
Comment thread CHANGELOG.md Outdated
Comment thread charts/puris/Chart.yaml
Copy link
Copy Markdown
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST left a comment

Choose a reason for hiding this comment

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

LGTM. thanks a lot!

@tom-rm-meyer-ISST tom-rm-meyer-ISST merged commit f447989 into eclipse-tractusx:main Oct 1, 2025
14 checks passed
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.

[Bug] All materials considered for projected stock in Material Details View

2 participants