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

do not block account menu: availability checking can be slow #3565

Merged
merged 1 commit into from
Jul 21, 2021

Conversation

mgallien
Copy link
Collaborator

ignore availability that can be very slow to check with CfApi virtual
files plugin

Signed-off-by: Matthieu Gallien matthieu.gallien@nextcloud.com

@FlexW
Copy link

FlexW commented Jul 20, 2021

Fixes #3567

@allexzander
Copy link
Contributor

@mgallien Nice. Just thought we could maybe just comment-out this code as we will be bringing it back later anyways.

@mgallien
Copy link
Collaborator Author

@mgallien Nice. Just thought we could maybe just comment-out this code as we will be bringing it back later anyways.

I prefer that we would do a revert like I did with the dynamic plugin loading code

@mgallien
Copy link
Collaborator Author

/rebase

@github-actions github-actions bot force-pushed the bugfix/fixSlowMenuInSettings branch from 6853125 to 35949d2 Compare July 20, 2021 14:51
@mgallien
Copy link
Collaborator Author

/rebase

@github-actions github-actions bot force-pushed the bugfix/fixSlowMenuInSettings branch from 35949d2 to e1867a5 Compare July 21, 2021 08:40
@mgallien
Copy link
Collaborator Author

/rebase

ignore availability that can be very slow to check with CfApi virtual
files plugin

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@github-actions github-actions bot force-pushed the bugfix/fixSlowMenuInSettings branch from e1867a5 to a8ef8bd Compare July 21, 2021 12:58
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-3565-a8ef8bdcb840cd4e474a71717e8c8392d7c23370-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.

@FlexW FlexW added this to the Desktop 3.3 milestone Jul 21, 2021
@mgallien mgallien merged commit b536b73 into master Jul 21, 2021
@mgallien mgallien deleted the bugfix/fixSlowMenuInSettings branch July 21, 2021 14:42
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.

4 participants