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

Groupware Clients Part 1 #923

Merged
merged 4 commits into from
Jan 14, 2019
Merged

Groupware Clients Part 1 #923

merged 4 commits into from
Jan 14, 2019

Conversation

MariusBluem
Copy link
Member

@MariusBluem MariusBluem commented Nov 6, 2018

@georgehrke @nextcloud/documentation

Signed-off-by: Marius Blüm marius@lineone.io

Signed-off-by: Marius Blüm <marius@lineone.io>
Signed-off-by: Marius Blüm <marius@lineone.io>
@MorrisJobke
Copy link
Member

In general the auto discovery works pretty well on iOS and macOS. I just entered username@cloud.example.org and my password and CalDAV and CardDav detected everything just fine.

It is only needed that the .well-known/carddav and .../caldav redirect works properly, which is shown in 14.0.3+ admin settings as a warning if broken.

@MariusBluem
Copy link
Member Author

In general the auto discovery works pretty well on iOS and macOS. I just entered username@cloud.example.org and my password and CalDAV and CardDav detected everything just fine.

You are right. I just wanted to document a streamlined way without alternatives which should work in every case, since we will have config-profiles for macOS and iOS in the future for easy setup. If you want, I will document both ways ...

@MorrisJobke
Copy link
Member

You are right. I just wanted to document a streamlined way without alternatives which should work in every case, since we will have config-profiles for macOS and iOS in the future for easy setup. If you want, I will document both ways ...

I would prefer the user friendly way of not need to fill out 5 different input fields if username + password would be enough ;)

@georgehrke
Copy link
Member

@MariusBluem What's the current status here?

Signed-off-by: Marius Blüm <marius@lineone.io>
Copy link

@bencomp bencomp left a comment

Choose a reason for hiding this comment

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

@MariusBluem I think you wrote mostly what I wanted to add, so great! My comments are really minor.

user_manual/pim/sync_osx.rst Outdated Show resolved Hide resolved
user_manual/pim/sync_osx.rst Outdated Show resolved Hide resolved
user_manual/pim/sync_osx.rst Outdated Show resolved Hide resolved
user_manual/pim/sync_osx.rst Outdated Show resolved Hide resolved
user_manual/pim/sync_osx.rst Outdated Show resolved Hide resolved
Signed-off-by: Marius Blüm <marius@lineone.io>
@MariusBluem MariusBluem changed the title [WIP] Groupware Documentation Groupware Documentation Jan 14, 2019
@MariusBluem MariusBluem changed the title Groupware Documentation Groupware Clients Jan 14, 2019
@MariusBluem MariusBluem changed the title Groupware Clients Groupware Clients Part 1 Jan 14, 2019
@MariusBluem
Copy link
Member Author

So we will bring this in now. It is an improvement and documents easier how to setup your calendar and contacts on macOS and iOS. Those instructions run stable on my personal iOS and macOS devices for more than 4 months now. Hope this will keep after the next update. Thank you very much for your help everybody.

@georgehrke @bencomp @MorrisJobke @schiessle @karlitschek @jospoortvliet

Please review and merge! Thanks :)

@MariusBluem
Copy link
Member Author

P.S: Next step which was planned was Windows 10 native client (not Outlook) but I've abandoned that since they are breaking it currently all the time :(


.. image:: ../images/synchronize_cardbook.png

The rest of the details of dealing with Thunderbird CardBook are left to the reader...
The rest of the details of dealing with Thunderbird CardBook are left to the reader...

Calendar
Copy link
Member

Choose a reason for hiding this comment

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

It would be nice to add that Thunderbird, unlike macOS, evolution or KOrganizer, does not support the automatic discovery of calendars. Hence you will have add each calendar, that you want to synchronise, individually.

Copy link
Member

Choose a reason for hiding this comment

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

Something for another PR as this is purely about whitespace changes 😉

@@ -62,4 +62,4 @@ https://cloud.nextcloud.com/remote.php/dav/calendars/daniel/personal/

.. image:: ../images/CalDAV_calendar.png

The rest of the details of dealing with Thunderbird Calendar are left to the reader...
The rest of the details of dealing with Thunderbird Calendar are left to the reader...
Copy link
Member

Choose a reason for hiding this comment

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

This sentence seems kind of weird to me. It kind of sounds like "We were to lazy to explain more. Go figure it out yourself, not our problem."

We explained how to connect Lightning to Nextcloud, so I would just replace this sentence with something like:
"Further instructions how to manage your calendars with Thunderbird Lightning can be found in their respecting documentation: https://wiki.mozilla.org/Calendar:User_Documentation"

Same goes for Cardbook above.

Copy link
Member

Choose a reason for hiding this comment

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

=> Another PR

@MorrisJobke MorrisJobke merged commit 6aa3a7f into master Jan 14, 2019
@MorrisJobke MorrisJobke deleted the groupware-sync branch January 14, 2019 13:02
@MorrisJobke
Copy link
Member

/backport to stable15

@MorrisJobke
Copy link
Member

/backport to stable14

@backportbot-nextcloud
Copy link

backport to stable15 in #1135

@backportbot-nextcloud
Copy link

backport to stable14 in #1136

@MariusBluem
Copy link
Member Author

@georgehrke

I will fix your comments as proposed by @MorrisJobke in another PR. Thanks!

@skjnldsv
Copy link
Member

skjnldsv commented Jan 15, 2019

@MariusBluem @MorrisJobke I think this pr caused the drone failure. Now tests don't pass 🤔

LaTeX Warning: File `{images/macos_1}.png' not found on input line 2365.


LaTeX Warning: File `{images/macos_1}.png' not found on input line 2365.


! Package pdftex.def Error: File `images/macos_1.png' not found: using draft se
tting.

See the pdftex.def package documentation for explanation.
Type  H <return>  for immediate help.
 ...                                              
                                                  
l.2365 ...inxincludegraphics{{images/macos_1}.png}
                                                  
? 
! Emergency stop.
 ...                                              
                                                  
l.2365 ...inxincludegraphics{{images/macos_1}.png}
                                                  
!  ==> Fatal error occurred, no output PDF file produced!

@skjnldsv
Copy link
Member

/drone/src/github.com/nextcloud/documentation/user_manual/pim/index.rst:14: WARNING: toctree contains reference to nonexisting document u'pim/troubleshooting'
/drone/src/github.com/nextcloud/documentation/user_manual/pim/sync_osx.rst:42: ERROR: Unknown directive type "figures".

.. figures:: /images/macos_4.png
/drone/src/github.com/nextcloud/documentation/user_manual/pim/sync_osx.rst:16: WARNING: image file not readable: images/macos_1.png
/drone/src/github.com/nextcloud/documentation/user_manual/pim/sync_osx.rst:21: WARNING: image file not readable: images/macos_2.png
/drone/src/github.com/nextcloud/documentation/user_manual/pim/sync_osx.rst:35: WARNING: image file not readable: images/macos_3.png
looking for now-outdated files... none found

@MariusBluem
Copy link
Member Author

🤔

@georgehrke
Copy link
Member

@MariusBluem The path needs to be ./images/macos_3.png, not /images/macos_3.png.

MorrisJobke added a commit that referenced this pull request Jan 16, 2019
*see #923

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke mentioned this pull request Jan 16, 2019
@MorrisJobke
Copy link
Member

@georgehrke @skjnldsv @MariusBluem Fix is in #1155

backportbot-nextcloud bot pushed a commit that referenced this pull request Jan 16, 2019
*see #923

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
backportbot-nextcloud bot pushed a commit that referenced this pull request Jan 16, 2019
*see #923

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants