Skip to content

Conversation

alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Sep 4, 2025

  • Tests written, or not not needed

Issue #15238

@alperozturk96 alperozturk96 linked an issue Sep 4, 2025 that may be closed by this pull request
}
}

private fun fetchFileAndRun(partialFile: OCFile, block: (file: OCFile) -> Unit) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unused code

mSyncInProgress = false
ocFileListFragment?.setLoading(false)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed for pull to refresh

@tobiasKaminsky
Copy link
Member

/rebase

@alperozturk96 alperozturk96 force-pushed the feat/do-not-show-loading-spinner-automatically branch from 08b0486 to 19e44f0 Compare September 9, 2025 10:19
Copy link

github-actions bot commented Sep 9, 2025

Codacy

Lint

TypemasterPR
Warnings4745
Errors1111

SpotBugs

CategoryBaseNew
Bad practice5959
Correctness6868
Dodgy code267267
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3535
Performance4848
Security1818
Total505505

@alperozturk96 alperozturk96 merged commit cfcf6e6 into master Sep 9, 2025
17 of 20 checks passed
@alperozturk96 alperozturk96 deleted the feat/do-not-show-loading-spinner-automatically branch September 9, 2025 12:12
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.

Hide loading spinner
2 participants