Skip to content

Changes for Nextcloud - #34

Merged
MorrisJobke merged 15 commits into
nextcloud:masterfrom
JSoko:master
Jun 22, 2016
Merged

MorrisJobke merged 15 commits into
nextcloud:masterfrom
JSoko:master

Conversation

@JSoko

@JSoko JSoko commented Jun 22, 2016 •

Copy link
Copy Markdown
Member

Changes for Nextcloud

multiple-user production users.

.. note:: ownCloud Enterprise edition does not support SQLite.
.. note:: Nextcloud Enterprise edition does not support SQLite.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is no Netcloud EE 😉 ...so we should remove this part completely.

@JSoko JSoko Jun 22, 2016 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

OK

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@JSoko You only fixed the other occurrence. This line here is still there. ;)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry I'm blind... Changed...

add_header X-Permitted-Cross-Domain-Policies none;

# Path to the root of your installation
root /var/www/owncloud/;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nextcloud

@MorrisJobke

Copy link
Copy Markdown
Member

@JSoko Thanks a lot. Beside my comments this looks really good :)

@JSoko

JSoko commented Jun 22, 2016

Copy link
Copy Markdown
Member Author

@MorrisJobke thank you for your work an for your comments... :-))

@MorrisJobke

Copy link
Copy Markdown
Member

Makes sense now :) Thanks a lot 👍

@MorrisJobke
MorrisJobke merged commit 7b27cf8 into nextcloud:master Jun 22, 2016
@JSoko

JSoko commented Jun 22, 2016

Copy link
Copy Markdown
Member Author

Pushed tham reworks more... #37

@MorrisJobke MorrisJobke mentioned this pull request Mar 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants