Skip to content

feat(Partner Center Sell): extending IAM and Deployment objects with new parameters#449

Merged
pyrooka merged 3 commits into
mainfrom
new-properties-for-deployment-and-iam-objects--parnercentersellv1
Jun 4, 2026
Merged

feat(Partner Center Sell): extending IAM and Deployment objects with new parameters#449
pyrooka merged 3 commits into
mainfrom
new-properties-for-deployment-and-iam-objects--parnercentersellv1

Conversation

@HarasztiaPeter

@HarasztiaPeter HarasztiaPeter commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

PR summary

Adding new parameter metadata.other. location_proxied_by to deployment objects
Adding supportedNetwork.operations.defaults option for IAM CBR
Adding eventPublishing option for CBR

PR Checklist

Please make sure that your PR fulfills the following requirements:

  • The commit message follows the Angular Commit Message Guidelines.
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

…to deployments, default values for cbr, also new parameter for them called event_publishing on the IAM service object

Signed-off-by: Peter Harasztia <peter.harasztia@ibm.com>
@HarasztiaPeter

Copy link
Copy Markdown
Contributor Author

Example tests
Screenshot 2026-06-02 at 14 16 42

@HarasztiaPeter

Copy link
Copy Markdown
Contributor Author

Integration tests:
Screenshot 2026-06-02 at 14 54 49

@HarasztiaPeter HarasztiaPeter requested a review from pyrooka June 2, 2026 13:07
@HarasztiaPeter HarasztiaPeter marked this pull request as ready for review June 2, 2026 13:07
@HarasztiaPeter HarasztiaPeter marked this pull request as draft June 2, 2026 14:00
Signed-off-by: Peter Harasztia <peter.harasztia@ibm.com>
@HarasztiaPeter HarasztiaPeter marked this pull request as ready for review June 2, 2026 14:09

@pyrooka pyrooka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Overall the PR looks good, but left a question to talk about before merging.

}

// DefaultServiceURL is the default URL to make service requests to.
const DefaultServiceURL = "https://product-lifecycle.cloud.ibm.com/openapi/v1"

@pyrooka pyrooka Jun 3, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Merely a question, but is this new default service URL correct? Using localhost doesn't seem right to me in this context, but please correct me if I'm wrong.

@CLAassistant

CLAassistant commented Jun 4, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@v1szk1s v1szk1s force-pushed the new-properties-for-deployment-and-iam-objects--parnercentersellv1 branch 2 times, most recently from 3dd2d63 to eaae460 Compare June 4, 2026 13:27
Signed-off-by: Attila Ambrus <aattila0@proton.me>
@v1szk1s v1szk1s force-pushed the new-properties-for-deployment-and-iam-objects--parnercentersellv1 branch from eaae460 to 89c08b8 Compare June 4, 2026 13:30
@pyrooka pyrooka changed the title feat(partner center sell): extending IAM and Deployment objects with new parameters feat(Partner Center Sell): extending IAM and Deployment objects with new parameters Jun 4, 2026

@pyrooka pyrooka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@pyrooka pyrooka merged commit be5979c into main Jun 4, 2026
12 checks passed
@pyrooka pyrooka deleted the new-properties-for-deployment-and-iam-objects--parnercentersellv1 branch June 4, 2026 14:35
ibm-devx-sdk pushed a commit that referenced this pull request Jun 4, 2026
# [0.99.0](v0.98.0...v0.99.0) (2026-06-04)

### Features

* **Partner Center Sell:** extending IAM and Deployment objects with new parameters ([#449](#449)) ([be5979c](be5979c))
@ibm-devx-sdk

Copy link
Copy Markdown

🎉 This PR is included in version 0.99.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants