Skip to content

Change the API prefix for the coreHttpApi module to /api/core/v1#627

Merged
jkoenig134 merged 8 commits intorelease/v7from
change-the-api-prefix-for-the-coreHttpApi-module
Aug 19, 2025
Merged

Change the API prefix for the coreHttpApi module to /api/core/v1#627
jkoenig134 merged 8 commits intorelease/v7from
change-the-api-prefix-for-the-coreHttpApi-module

Conversation

@jkoenig134
Copy link
Copy Markdown
Contributor

@jkoenig134 jkoenig134 commented Aug 19, 2025 •
edited
Loading

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.
  • I self-reviewed the PR.

@jkoenig134 jkoenig134 added the breaking-change A breaking change label Aug 19, 2025
@jkoenig134 jkoenig134 changed the base branch from main to release/v7 August 19, 2025 09:28
@jkoenig134 jkoenig134 changed the title Change the API prefix for the coreHttpApi module Change the API prefix for the coreHttpApi module to /api/core/v1 Aug 19, 2025
@jkoenig134 jkoenig134 enabled auto-merge (squash) August 19, 2025 09:32
britsta
britsta reviewed Aug 19, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@britsta britsta left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Is the apiVersion of the Chart.yaml still v2?

@jkoenig134
Copy link
Copy Markdown
Contributor Author

jkoenig134 commented Aug 19, 2025 •
edited
Loading

Is the apiVersion of the Chart.yaml still v2?

Ehm, that's helm syntax and has nothing to do with the connectors api :)

@jkoenig134 jkoenig134 requested a review from britsta August 19, 2025 11:17
@jkoenig134 jkoenig134 merged commit 8555dfe into release/v7 Aug 19, 2025
8 checks passed
@jkoenig134 jkoenig134 deleted the change-the-api-prefix-for-the-coreHttpApi-module branch August 19, 2025 12:06
This was referenced Aug 19, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@britsta britsta britsta approved these changes

@tnotheis tnotheis Awaiting requested review from tnotheis

@Milena-Czierlinski Milena-Czierlinski Awaiting requested review from Milena-Czierlinski

Assignees

No one assigned

Labels

breaking-change A breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jkoenig134 @britsta