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

Allow to change preferences with machine token #12637

Merged
merged 1 commit into from
Feb 12, 2019

Conversation

mmorhun
Copy link
Contributor

@mmorhun mmorhun commented Feb 8, 2019

What does this PR do?

This PR allows to change (update / replace / delete) user preferences with machine token.
This is done to be able to change user preferences from within a workspace container.

What issues does this PR fix or reference?

#12442

Signed-off-by: Mykola Morhun <mmorhun@redhat.com>
@che-bot che-bot added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/enhancement A feature request - must adhere to the feature request template. labels Feb 8, 2019
@mmorhun mmorhun added kind/task Internal things, technical debt, and to-do tasks to be performed. and removed kind/enhancement A feature request - must adhere to the feature request template. labels Feb 8, 2019
@skabashnyuk
Copy link
Contributor

@mmorhun I think we should notify community on email about this change. If nobody will raise any concerns I'm +1.

Copy link
Member

@sleshchenko sleshchenko left a comment

Choose a reason for hiding this comment

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

Previously, there were available the only workspace related methods or read operation for other API.
With these changes, it is possible with one machine token to modify other workspaces data.
But it's OK for me since there is no more suitable service for storing data by IDE from workspace container.

@mmorhun mmorhun merged commit e2991cd into master Feb 12, 2019
@mmorhun mmorhun deleted the allow-change-preferences-with-machine-token branch February 12, 2019 12:05
@che-bot che-bot added this to the 6.19.0 milestone Feb 12, 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 Feb 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants