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

[stable-3.0] Allow server URL to be pre-defined without enforcing it #2465

Merged
merged 1 commit into from Sep 23, 2020

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #2441

…modified

APPLICATION_SERVER_URL can be either empty or be specified. This commit adds the new CMake option APPLICATION_SERVER_URL_ENFORCE to decide whether to enforce the
URL's unmodified use (like before, default: ON) or to allow modification by the user (new).

By default APPLICATION_SERVER_URL_ENFORCE is set to ON, to no break with the previous implementation's expectations.

If APPLICATION_SERVER_URL is empty, APPLICATION_SERVER_URL_ENFORCE will be ignored by the Account Wizard.

The previous behaviour confused me a bit with branded builds. When the URL was (usually) specified but not forced, it was simply discarded, forcing the user to
manually supply it.

Signed-off-by: Michael Schuster <michael@schuster.ms>
@backportbot-nextcloud backportbot-nextcloud bot added enhancement enhancement of a already implemented feature/code feature: ⚙️ settings Settings panel functionalities. labels Sep 23, 2020
@backportbot-nextcloud backportbot-nextcloud bot added this to the Desktop 3.0 milestone Sep 23, 2020
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-2465-25c8077378ba13e8fc6168e99479aa2941ad8cf1-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@misch7 misch7 merged commit a909ffc into stable-3.0 Sep 23, 2020
@misch7 misch7 deleted the backport/2441/stable-3.0 branch September 23, 2020 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement enhancement of a already implemented feature/code feature: ⚙️ settings Settings panel functionalities.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants