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(platform): server:start with specified domain #239

Merged
merged 1 commit into from
Aug 2, 2019

Conversation

sleshchenko
Copy link
Contributor

@sleshchenko sleshchenko commented Aug 1, 2019

What does this PR do?

Do not ignore the domain parameter if it's specified.

What issues does this PR fix or reference?

eclipse-che/che#14095

Signed-off-by: Sergii Leshchenko <sleshche@redhat.com>
@benoitf
Copy link
Collaborator

benoitf commented Aug 1, 2019

waiting target milestone of referenced issue

@sleshchenko sleshchenko changed the title fix(platform): server:start with specified domain WIP fix(platform): server:start with specified domain Aug 1, 2019
@sleshchenko
Copy link
Contributor Author

@benoitf Do you think it makes sense to enable WIP bot for chectl repo?
That you would be able to approve if you're OK with changes, but it would not be merged without a milestone.

@benoitf
Copy link
Collaborator

benoitf commented Aug 1, 2019

@sleshchenko done

@ibuziuk
Copy link
Contributor

ibuziuk commented Aug 1, 2019

@benoitf @sleshchenko are you planning to merge it before or after GA ?

@sleshchenko
Copy link
Contributor Author

@ibuziuk Since the referenced issue is labeled with 7.0.0 milestone, I can merge it right now.

@sleshchenko sleshchenko changed the title WIP fix(platform): server:start with specified domain fix(platform): server:start with specified domain Aug 2, 2019
@sleshchenko sleshchenko merged commit 5ab0061 into che-incubator:master Aug 2, 2019
@sleshchenko sleshchenko deleted the fixDomainParam branch August 2, 2019 06:38
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