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

Fix - Modified invocation of the putArrangementById API to provide the internal ID of the arrangement instead of the external ID #486

Closed

Conversation

joshua-backbase
Copy link
Contributor

@joshua-backbase joshua-backbase commented Nov 18, 2024

Description

Fix - Modified invocation of the putArrangementById API to provide the internal ID of the arrangement instead of the external ID

Checklist

  • I made sure, I read CONTRIBUTING.md to put right branch prefix as per my need.
  • I made sure to update CHANGELOG.md.
  • I made sure to update Stream Wiki(only valid in case of new stream module or architecture changes).
  • My changes are adequately tested.
  • I made sure all the SonarCloud Quality Gate are passed.

@joshua-backbase joshua-backbase requested a review from a team as a code owner November 18, 2024 20:07
…e internal ID of the arrangement instead of the external ID
…e internal ID of the arrangement instead of the external ID
…e internal ID of the arrangement instead of the external ID
@joshua-backbase joshua-backbase force-pushed the hotfix/fix-put-arrangement-by-id-api-call branch from 121ac87 to 1edb5a4 Compare November 19, 2024 13:48
auto-merge was automatically disabled November 19, 2024 13:54

Pull request was closed

@joshua-backbase joshua-backbase deleted the hotfix/fix-put-arrangement-by-id-api-call branch November 19, 2024 13:54
Copy link

sonarcloud bot commented Nov 19, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
16.7% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

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

Successfully merging this pull request may close these issues.

3 participants