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

Bug 1492991 - Remove alpha prefix from service catalog properties #2113

Merged
merged 1 commit into from
Sep 19, 2017

Conversation

spadgett
Copy link
Member

@spadgett spadgett commented Sep 19, 2017

https://trello.com/c/y1lbFrbp

Bump origin-web-catalog to 0.0.47
Depends on openshift/origin-web-catalog#442

Since service-catalog 0.0.18, the alpha prefix has been removed from all fields in favor of feature gating. Make the corresponding API changes in the web catalog.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1492991

Copy link
Member

@jeff-phillips-18 jeff-phillips-18 left a comment

Choose a reason for hiding this comment

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

LGTM. I could not find any other occurrences.

Since service-catalog 0.0.18, the alpha prefix has been removed from all
fields in favor of feature gating. Make the corresponding API changes in
the web catalog.
@spadgett
Copy link
Member Author

Bumped origin-web-catalog to 0.0.47 with the same API changes and added dist.

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to 615bdb8

@openshift-bot
Copy link

openshift-bot commented Sep 19, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin_web_console/224/) (Base Commit: e76c041) (PR Branch Commit: 615bdb8)

@openshift-bot openshift-bot merged commit 6f40623 into openshift:master Sep 19, 2017
@spadgett spadgett deleted the remove-alpha-prefix branch September 19, 2017 14:02
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.

3 participants