Skip to content

Bug fixes#800

Merged
tom-rm-meyer-ISST merged 5 commits intoeclipse-tractusx:mainfrom
achtzig20:fix/partner-selection
Feb 26, 2025
Merged

Bug fixes#800
tom-rm-meyer-ISST merged 5 commits intoeclipse-tractusx:mainfrom
achtzig20:fix/partner-selection

Conversation

@ReneSchroederLJ
Copy link
Copy Markdown
Member

@ReneSchroederLJ ReneSchroederLJ commented Feb 26, 2025

Description

  • fixed an error in retrieving the correct list of partners when creating a delivery
  • added missing mandatory field indicators
  • changed the naming of "Supplier Site" to "Expected Supplier Site" in the add demand modal
  • fixed translation of weekdays to english language

resolves #417, #421, #429, #523, #798, #799

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).

@ReneSchroederLJ ReneSchroederLJ marked this pull request as ready for review February 26, 2025 12:26
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, when walking through your fixes, I noticed that the #798 was a duplicate of #421

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