Skip to content

BugFix - Prevent Unnecessary Gallery Binding#14692

Merged
tobiasKaminsky merged 8 commits intomasterfrom
bugfix/gallery-unnecessary-rebind
Apr 16, 2025
Merged

BugFix - Prevent Unnecessary Gallery Binding#14692
tobiasKaminsky merged 8 commits intomasterfrom
bugfix/gallery-unnecessary-rebind

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Mar 17, 2025

  • Tests written, or not not needed

Copy link
Member

@tobiasKaminsky tobiasKaminsky left a comment

Choose a reason for hiding this comment

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

as written inline

Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 force-pushed the bugfix/gallery-unnecessary-rebind branch from f5bf09a to ae0e9be Compare April 11, 2025 07:35
Signed-off-by: alperozturk <alper_ozturk@proton.me>
…s no need do the check

Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings5454
Errors1111

SpotBugs

CategoryBaseNew
Bad practice6565
Correctness5858
Dodgy code292292
Experimental11
Internationalization77
Malicious code vulnerability11
Multithreaded correctness77
Performance5151
Security1818
Total500500

@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/14692.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@tobiasKaminsky tobiasKaminsky merged commit fcab4c1 into master Apr 16, 2025
20 checks passed
@tobiasKaminsky tobiasKaminsky deleted the bugfix/gallery-unnecessary-rebind branch April 16, 2025 09:30
@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 3.32.0 milestone May 8, 2025
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.

2 participants