Skip to content

Conversation

@palenhart
Copy link

nextcloud/documentation@e917b2a seems to be generated from this file and reverted all occurrences of Nextcloud back to ownCloud.
I replaced this config sample with the config sample from the master branch.

It would be great if someone could check whether all changes to available parameters from master are also true for stable9 and the changes in my commit are correct 😄 @MorrisJobke @LukasReschke

@MariusBluem MariusBluem added the 3. to review Waiting for reviews label Jun 23, 2016
@MariusBluem MariusBluem added this to the Nextcloud 9.0.52 milestone Jun 23, 2016
*
* Updating/Deleting this value can make connected clients stall until
* the user has resolved conflicts.
*/
Copy link
Member

@LukasReschke LukasReschke Jun 26, 2016

Choose a reason for hiding this comment

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

Not relevant for stable9

@LukasReschke
Copy link
Member

LGTM besides my small remark 🚀

* Enforce token authentication for clients, which blocks requests using the user
* password for enhanced security. Users need to generate tokens in personal settings
* which can be used as passwords on their clients.
*/
Copy link
Member

Choose a reason for hiding this comment

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

This is not available in stable9. Could you remove it?

@MorrisJobke
Copy link
Member

@zollak Thanks a lot :) Once my remarks are fixed this can get merged :)

@palenhart
Copy link
Author

Thank you for reviewing 😄
I removed all the non-available options.

@MorrisJobke
Copy link
Member

Nice 👍

@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 26, 2016
@MorrisJobke MorrisJobke merged commit 62391e9 into nextcloud:stable9 Jun 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants