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

show server url to connect. Not everyone know that this can be copied… #176

Merged
merged 1 commit into from
Apr 10, 2019

Conversation

karlitschek
Copy link
Member

@karlitschek karlitschek commented Mar 19, 2019

… from the browser url bar
Signed-off-by: Frank Karlitschek frank@nextcloud.com

Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

All file permissions seem to have changed from 644 to 755

@karlitschek
Copy link
Member Author

@nickvergessen fixed. Could you have a look? Would be great to get this simple improvement in :-)

@skjnldsv skjnldsv requested a review from nickvergessen April 10, 2019 07:32
@skjnldsv skjnldsv force-pushed the showserverurl branch 2 times, most recently from 1df0f6a to ff0c955 Compare April 10, 2019 07:36
@skjnldsv skjnldsv requested a review from nickvergessen April 10, 2019 07:36
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

🐘

@skjnldsv skjnldsv force-pushed the showserverurl branch 2 times, most recently from 5e93372 to 02b01f7 Compare April 10, 2019 08:18
Not everyone know that this can be copied from the browser url    bar

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@skjnldsv skjnldsv merged commit 6acfd6e into master Apr 10, 2019
@skjnldsv skjnldsv deleted the showserverurl branch April 10, 2019 08:27
@rullzer rullzer mentioned this pull request Apr 10, 2019
13 tasks
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