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

Update README basing on issue #596 #617

Merged
merged 1 commit into from
Nov 3, 2022
Merged

Conversation

cm-schl
Copy link

@cm-schl cm-schl commented Nov 3, 2022

Checked the README and updated the screenshots. Also added a detailed description how the automatic login options work (basing on #596).

I'm not sure if SnappyMail is always first trying to use the user defined credentials and then the other options (showing prompt / passing Nextcloud username / passing Nextcloud mailaddress) - but to me it seemed the only logical procedure. Please correct me if I'm wrong.

I've leaved the paragraphs "Auto-connection for all Nextcloud users" and "Auto-connection for one user at a time" because I unfortunately didn't understand if these are talking about the same tree options in the Nextcloud "Additional settings" or some other config options...

Checked the README and updated the screenshots. Also added a detailed description how the automatic login options work (basing on the-djmaze#596).

I'm not sure if SnappyMail is always first trying to use the user defined credentials and then the other options (showing prompt / passing Nextcloud username / passing Nextcloud mailaddress) - but to me it seemed the only logical procedure. Please correct me if I'm wrong.

I've leaved the paragraphs "Auto-connection for all Nextcloud users" and "Auto-connection for one user at a time" because I unfortunately didn't understand if these are talking about the same tree options in the Nextcloud "Additional settings" or some other config options...
@the-djmaze the-djmaze merged commit 61e142b into the-djmaze:master Nov 3, 2022
@cm-schl cm-schl deleted the patch-2 branch November 9, 2022 11:02
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.

2 participants