Skip to content

fix(batch frontend): fix typo in ENDPOINT_PARTNER_DATA_UPDATE_BATCH and wrong var in chart#1114

Merged
tom-rm-meyer-ISST merged 1 commit intomainfrom
fix/chart
Mar 9, 2026
Merged

fix(batch frontend): fix typo in ENDPOINT_PARTNER_DATA_UPDATE_BATCH and wrong var in chart#1114
tom-rm-meyer-ISST merged 1 commit intomainfrom
fix/chart

Conversation

@tom-rm-meyer-ISST
Copy link
Copy Markdown
Contributor

@tom-rm-meyer-ISST tom-rm-meyer-ISST commented Mar 9, 2026

Description

Two errors identified:

  • helm frontend puris used VITE prefix
  • all variables had a typo (ENDPOINT_PARNTER_DATA_UPDATE_BATCH instead of ENDPOINT_PARTNER_DATA_UPDATE_BATCH

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 (TRG 7.02
  • Documentation Notice are present on all affected files (TRG 7.07)
  • If helm chart has been changed, the chart version has been bumped to either next major, minor or patch level (compared to released chart).
  • Changelog updated (changelog.md) with PR reference and brief summary.
  • Frontend version bumped, if needed (frontend/package.json, frontend/package-lock.json)
  • Backend version bumped, if needed (backend/pom.xml)
  • Open API specification updated, if controllers have been changed (use python script scripts/generate_openapi_yaml.py with running customer backend)

@tom-rm-meyer-ISST tom-rm-meyer-ISST changed the title - Add missing frontend endpoint path in chart ([#112](https://github.com/eclipse-tractusx/puris/pull/1112)) fix(batch frontend): fix typo in ENDPOINT_PARTNER_DATA_UPDATE_BATCH and wrong var in chart Mar 9, 2026
@tom-rm-meyer-ISST tom-rm-meyer-ISST marked this pull request as ready for review March 9, 2026 10:17
Copy link
Copy Markdown
Member

@ReneSchroederLJ ReneSchroederLJ left a comment

Choose a reason for hiding this comment

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

LGTM. good catch. My apologies for not catching this during earlier reviews

@tom-rm-meyer-ISST tom-rm-meyer-ISST merged commit 119fa06 into main Mar 9, 2026
13 of 14 checks passed
@tom-rm-meyer-ISST tom-rm-meyer-ISST deleted the fix/chart branch March 9, 2026 10:28
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.

2 participants