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.

Sign up for GitHub

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

Jump to bottom

[stable12] Don't allow the user to set fields they can't see #5270

Merged
merged 3 commits into from
Jun 6, 2017
Merged

[stable12] Don't allow the user to set fields they can't see #5270

merged 3 commits into from
Jun 6, 2017

Conversation

MorrisJobke
Copy link
Member

@nickvergessen @MorrisJobke
Don't allow the user to set fields they can't see
ce34db2 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @MorrisJobke
Also cover the settings endpoint
a62b292 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen @MorrisJobke
adjust the test
b39dfe8 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@MorrisJobke MorrisJobke added the 3. to review Waiting for reviews label Jun 6, 2017
@MorrisJobke MorrisJobke added this to the Nextcloud 12.0.1 milestone Jun 6, 2017
@schiessle schiessle merged commit f29d53f into stable12 Jun 6, 2017
@schiessle schiessle deleted the stable12-5223 branch June 6, 2017 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes

@schiessle schiessle schiessle approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

Assignees
No one assigned
Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
Nextcloud 12.0.1
Development

Successfully merging this pull request may close these issues.
3 participants
@MorrisJobke @nickvergessen @schiessle