Skip to content

refactor(web-view-util): use proxy controller - #17850

Open
alperozturk96 wants to merge 2 commits into
masterfrom
refactor/web-view-util
Open

alperozturk96 wants to merge 2 commits into
masterfrom
refactor/web-view-util

Conversation

@alperozturk96

@alperozturk96 alperozturk96 commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Changes

  • Instead of using reflection we can use ProxyController.
  • Separate concerns between files.
  • Removes dead codes.

@alperozturk96 alperozturk96 added the ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) label Oct 20, 2026
@alperozturk96

alperozturk96 commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator Author

@tobiasKaminsky how can we test this?

@codecov

codecov Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice2525
Correctness5252
Dodgy code161161
Experimental11
Malicious code vulnerability11
Multithreaded correctness2020
Performance2828
Security1616
Total304304

@github-actions

github-actions Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

📱 QA build

Download app-qa-debug.apk
QR code Open the QR code for this download
Commit ff9a947
Version 17850
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.

@alperozturk96

Copy link
Copy Markdown
Collaborator Author

@daniele-verducci you can directly use has proxy check inside the where it used thus no need extension since only one place using it.

alperozturk96 and others added 2 commits October 21, 2026 11:50
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: daniele-verducci <daniele.verducci@nextcloud.com>
@github-actions

Copy link
Copy Markdown

stable22-IT test failed, but no output was generated. Maybe a preliminary stage failed.

This branch has not been deployed

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

Labels

2. developing ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants