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

Add OSX fix for contacts sync #1281

Closed
wants to merge 2 commits into from
Closed

Add OSX fix for contacts sync #1281

wants to merge 2 commits into from

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Mar 2, 2019

@skjnldsv skjnldsv self-assigned this Mar 2, 2019
@MariusBluem
OSX is now called macOS
10d2948 
Signed-off-by: Marius Blüm <[email protected]>
MariusBluem
MariusBluem approved these changes Mar 2, 2019
View reviewed changes
Copy link
Member

@MariusBluem MariusBluem left a comment

Choose a reason for hiding this comment

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

👍 with my last commit

@skjnldsv
Copy link
Member Author

skjnldsv commented Mar 2, 2019 •
edited
Loading

Thanks a lot @MariusBluem :)
There is a reference forum thread, shall we link it from the docs? https://help.nextcloud.com/t/high-sierra-address-book-does-not-sync/23007

@BernieO
Copy link
Contributor

BernieO commented Mar 3, 2019 •
edited
Loading

This doesn't work for me - and I doubt that this is a solution for Mac OS X not being able to sync more then one addressbook.
For details see my comment here:
nextcloud/contacts#399 (comment)

Additionally: the to be commented outerror_page directives are anyway not present in the nginx configuration (not in latest, stable, 15, 14 and also not in 13)

MorrisJobke
MorrisJobke reviewed Mar 4, 2019
View reviewed changes
admin_manual/installation/nginx.rst
error_page 403 /core/templates/403.php;
error_page 404 /core/templates/404.php;

2. Change ``fastcgi_intercept_errors`` from ``on`` to ``off``.
Copy link
Member

Choose a reason for hiding this comment

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

Those files don't exist anymore since Nextcloud 12. 👎

@skjnldsv skjnldsv closed this Mar 4, 2019
@skjnldsv
Copy link
Member Author

skjnldsv commented Mar 4, 2019

Closing as it apparently doesn't fix anything :(

@skjnldsv skjnldsv deleted the osx-contacts-sync branch March 4, 2019 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MorrisJobke MorrisJobke MorrisJobke left review comments

@MariusBluem MariusBluem MariusBluem approved these changes

@georgehrke georgehrke Awaiting requested review from georgehrke

Assignees

@skjnldsv skjnldsv

Labels
2. developing manual: admin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@skjnldsv @BernieO @MorrisJobke @MariusBluem