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.

Sign up for GitHub

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

Jump to bottom

Fetch capabilities before first sync #11168

Closed
TheOneRing opened this issue Aug 31, 2023 · 0 comments · Fixed by #11187
Closed

Fetch capabilities before first sync #11168

TheOneRing opened this issue Aug 31, 2023 · 0 comments · Fixed by #11187
Assignees
@TheOneRing
Labels
p2-high Escalation, on top of current planning, release blocker type:bug
Milestone
Desktop 5.0

Comments

@TheOneRing
Copy link
Contributor

TheOneRing commented Aug 31, 2023 •
edited
Loading

We cache the capabilities in the settings file to have them around during startup.
On first connect we should refresh them BEFORE we start a sync.
@TheOneRing TheOneRing added Placeholder type:bug p2-high Escalation, on top of current planning, release blocker and removed Placeholder labels Aug 31, 2023
@TheOneRing TheOneRing added this to the Desktop 5.0 milestone Aug 31, 2023
@TheOneRing TheOneRing self-assigned this Sep 6, 2023
TheOneRing added a commit that referenced this issue Sep 6, 2023
@TheOneRing
Only sync once the capabilities where fetched
414a887 
Fixes: #11168
@TheOneRing TheOneRing mentioned this issue Sep 6, 2023
Merged
@TheOneRing TheOneRing linked a pull request Sep 6, 2023 that will close this issue
Only sync once the capabilities where fetched #11187
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TheOneRing TheOneRing

Labels
p2-high Escalation, on top of current planning, release blocker type:bug
Projects
None yet
Milestone
Desktop 5.0
Development

Successfully merging a pull request may close this issue.

Only sync once the capabilities where fetched owncloud/client
1 participant
@TheOneRing