Skip to content

fix(offline-operation): actions - #17644

Open
alperozturk96 wants to merge 8 commits into
masterfrom
fix/offline-operation-action-on-slow-network
Open

alperozturk96 wants to merge 8 commits into
masterfrom
fix/offline-operation-action-on-slow-network

Conversation

@alperozturk96

Copy link
Copy Markdown
Collaborator

Steps to reproduce

  1. Have a slow connection
  2. Try to create folder when device is in airplane mode
  3. Activate slow connection
  4. When user taps the retry button it should trigger the offline operation action and successfully complete the operation

Changes

  • Fixes file list refresh for adding offline operation
  • Fixes offline operation worker logic for early exit and remote file existence check against the stored offline operation
  • Distinguishes walled and reachability timing so that walled connection can be recovered earlier than 5 minutes

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
…ent unnecessary job

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
…d reachability cache timing

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3232
Correctness5858
Dodgy code186186
Experimental11
Malicious code vulnerability22
Multithreaded correctness2626
Performance3737
Security1919
Total361361

@github-actions

Copy link
Copy Markdown

📱 QA build

Download app-qa-debug.apk (76M)
QR code Open the QR code for this download
Commit 51e6f64
Version 17644
Available until 7 days after this build

The QA build installs alongside a released Nextcloud app, so you can keep
using your existing install while testing.

Downloading the file requires a GitHub account, so open this link on the
device you want to test on, or transfer the APK to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant