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 ability to configure workspace ephemeral mode #12483

Merged
merged 1 commit into from
Jan 23, 2019
Merged

Conversation

ashumilova
Copy link
Contributor

Signed-off-by: Anna Shumilova [email protected]

What does this PR do?

Adds ability to configure workspace ephemeral mode for workspace on Overview page.
screenshot from 2019-01-21 15-58-10
screenshot from 2019-01-21 15-58-01

What issues does this PR fix or reference?

#12374

Release Notes

Docs PR

@ashumilova ashumilova self-assigned this Jan 21, 2019
@ashumilova ashumilova requested a review from benoitf as a code owner January 21, 2019 14:03
@ashumilova ashumilova added the kind/enhancement A feature request - must adhere to the feature request template. label Jan 21, 2019
@che-bot che-bot added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 21, 2019
@ashumilova
Copy link
Contributor Author

ci-test

@che-bot
Copy link
Contributor

che-bot commented Jan 21, 2019

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:12483
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@ashumilova
Copy link
Contributor Author

ci-build

onEphemeralModeChange(): void {
debugger;
if (this.isEphemeralMode) {
this.workspaceDetails.config.attributes.persistVolumes = true + '';
Copy link
Contributor

Choose a reason for hiding this comment

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

'true' directly ?

this.workspaceDetails.config.attributes.persistVolumes = true + '';
} else {
if (this.attributesCopy.persistVolumes) {
this.workspaceDetails.config.attributes.persistVolumes = false + '';
Copy link
Contributor

Choose a reason for hiding this comment

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

'false' ?
false.toString() ?

@ashumilova
Copy link
Contributor Author

ci-test

@che-bot
Copy link
Contributor

che-bot commented Jan 22, 2019

Results of automated E2E tests of Eclipse Che Multiuser on OCP:
Build details
Test report
docker image: eclipseche/che-server:12483
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

@ashumilova ashumilova merged commit 1e1f790 into master Jan 23, 2019
@ashumilova ashumilova deleted the che-12374 branch January 23, 2019 08:32
@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 Jan 23, 2019
@che-bot che-bot added this to the 6.18.0 milestone Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants