Skip to content

feat(Context-Based Restrictions): add service group support to Operations #219

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

Merged
merged 7 commits into from
Sep 15, 2023

Conversation

JonahFarc
Copy link
Contributor

PR summary

Adds service group support to operations

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

service_name parameter can now be replaced with the service group parameter.

Does this PR introduce a breaking change?

  • Yes
  • No

@CLAassistant
Copy link

CLAassistant commented Sep 13, 2023

CLA assistant check
All committers have signed the CLA.

@padamstx padamstx self-requested a review September 13, 2023 16:56
@padamstx padamstx self-assigned this Sep 13, 2023
Copy link
Member

@soohoowoohoo soohoowoohoo left a comment

Choose a reason for hiding this comment

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

One comment to not undo items needed for example generation.

Comment on lines 47 to 53
account_id = None
service_name = None
vpc_crn = None
zone_id = None
zone_rev = None
rule_id = None
rule_rev = None
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we want to remove these values. You'll need them for the example regeneration. This applies to basically every change in this file.

Signed-off-by: Dylan <[email protected]>
Copy link
Member

@soohoowoohoo soohoowoohoo left a comment

Choose a reason for hiding this comment

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

LGTM

@padamstx
Copy link
Member

@JonahFarc Dylan, looks like there were lint errors in the Travis build. Please run make lint-fix in your sandbox (or just fix the one problem that was identified).

Copy link
Member

@padamstx padamstx left a comment

Choose a reason for hiding this comment

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

Looks good except for the lint error :)

Signed-off-by: Dylan <[email protected]>
Copy link
Member

@padamstx padamstx left a comment

Choose a reason for hiding this comment

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

LGTM

@padamstx padamstx merged commit 8b666db into IBM:main Sep 15, 2023
ibm-devx-sdk pushed a commit that referenced this pull request Sep 15, 2023
# [0.43.0](v0.42.0...v0.43.0) (2023-09-15)

### Features

* **Context-Based Restrictions:** add service group support to operations ([#219](#219)) ([8b666db](8b666db))
@ibm-devx-sdk
Copy link

🎉 This PR is included in version 0.43.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@JonahFarc JonahFarc deleted the cbr-update branch September 15, 2023 20:29
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