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

Add platform API prefix #32

Merged
merged 3 commits into from
May 21, 2024
Merged

Add platform API prefix #32

merged 3 commits into from
May 21, 2024

Conversation

NiraimathiGunalan
Copy link
Collaborator

@NiraimathiGunalan NiraimathiGunalan commented May 20, 2024

Copy link
Collaborator

@ruairi-hayes ruairi-hayes 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 @NiraimathiGunalan . Just a small comment on the wording

README.md Outdated
@@ -13,6 +13,7 @@ See [action.yml](action.yml)
The following parameters are always required:

- API_HOST - Domain name of the ApiConnect instance where discovered APIs will be sent.<br /> &nbsp; Example : `us-east.apiconnect.automation.ibm.com`
- PLATFORM_API_PREFIX - Platform API prefix have a default value of `platform-api` as same as for ApiConnect on Cloud. It can be changed to the env prefix
Copy link
Collaborator

Choose a reason for hiding this comment

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

....has a default......which is the same as the prefix value for ApiConnect on Cloud. It can be changed to match your systems set up if different from the default

@NiraimathiGunalan NiraimathiGunalan merged commit 27ddafd into main May 21, 2024
2 checks passed
@NiraimathiGunalan NiraimathiGunalan deleted the ng/prefix branch May 21, 2024 09:45
@NiraimathiGunalan NiraimathiGunalan restored the ng/prefix branch May 21, 2024 10:39
@NiraimathiGunalan NiraimathiGunalan deleted the ng/prefix branch May 21, 2024 10:39
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