Skip to content

Edit locally. Restart current folder sync immediately after file opened.#5588

Merged
allexzander merged 1 commit into
masterfrom
feature/restart-folder-sync
Apr 11, 2023
Merged

Edit locally. Restart current folder sync immediately after file opened.#5588
allexzander merged 1 commit into
masterfrom
feature/restart-folder-sync

Conversation

@allexzander

Copy link
Copy Markdown
Contributor

Make sure a folder sync gets restarted immediately after file opening is finished. A folder sync is restarted only if it was syncing before EditLocally got triggered.

@allexzander allexzander force-pushed the feature/restart-folder-sync branch from d2acd80 to e3e59bf Compare April 7, 2023 17:38
@allexzander

Copy link
Copy Markdown
Contributor Author

/backport to stable-3.8

@allexzander allexzander force-pushed the feature/restart-folder-sync branch from e3e59bf to 54c13a5 Compare April 11, 2023 07:50
@allexzander

Copy link
Copy Markdown
Contributor Author

/backport to stable-3.8

@nextcloud-desktop-bot

Copy link
Copy Markdown

AppImage file: nextcloud-PR-5588-54c13a560d9c093f122e543a8a6af08d2aeb71d7-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.

@sonarqubecloud

Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@codecov

codecov Bot commented Apr 11, 2023

Copy link
Copy Markdown

Codecov Report

Merging #5588 (54c13a5) into master (d4f5081) will decrease coverage by 0.02%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5588      +/-   ##
==========================================
- Coverage   59.27%   59.25%   -0.02%     
==========================================
  Files         143      143              
  Lines       18435    18435              
==========================================
- Hits        10927    10924       -3     
- Misses       7508     7511       +3     

see 1 file with indirect coverage changes

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