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 editor components alias attribute format to match plugins attribute format #14715

Merged
merged 1 commit into from
Sep 30, 2019

Conversation

mshaposhnik
Copy link
Contributor

@mshaposhnik mshaposhnik commented Sep 30, 2019

What does this PR do?

Fixes format of the id=alias editor component attribute to match the same for plugin components.
Fixes bug when the alias of editor component set via reference not reflected in the runtime machines.

What issues does this PR fix or reference?

Release Notes

N/A

Docs PR

N/A

…te format;

Signed-off-by: Max Shaposhnik <mshaposh@redhat.com>
@mshaposhnik mshaposhnik requested review from skabashnyuk and removed request for nickboldt, l0rd and rhopp September 30, 2019 13:35
@che-bot che-bot added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Sep 30, 2019
@che-bot
Copy link
Contributor

che-bot commented Sep 30, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

  • build details
  • "che-server" docker image: maxura/che-server:14715

@che-bot
Copy link
Contributor

che-bot commented Sep 30, 2019

E2E tests of Eclipse Che Multiuser on OCP has failed:

@mshaposhnik mshaposhnik merged commit ed148c2 into master Sep 30, 2019
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 30, 2019
@che-bot che-bot added this to the 7.3.0 milestone Sep 30, 2019
@mshaposhnik mshaposhnik deleted the alias_fix branch February 18, 2020 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants