Skip to content

refactor(files_external): Drop unused oauth1 code #54328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
skjnldsv merged 7 commits into from
Aug 8, 2025
Merged

refactor(files_external): Drop unused oauth1 code #54328

skjnldsv merged 7 commits into from
Aug 8, 2025

Conversation

@joshtrichards
Copy link
Member

@joshtrichards joshtrichards commented Aug 8, 2025 •
edited
Loading

  • Resolves: #

Summary

🧹 The oauth1 support was part of long ago removed Dropbox support in files_external. Other oauth1 pieces were removed in #6146, but a few remnants were left around (which this PR removes once and for all).

TODO

  • ...

Checklist

@joshtrichards joshtrichards added feature: external storage ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) 3. to review Waiting for reviews labels Aug 8, 2025
@joshtrichards joshtrichards marked this pull request as ready for review August 8, 2025 03:08
@joshtrichards joshtrichards requested review from a team as code owners August 8, 2025 03:08
@joshtrichards joshtrichards requested review from nfebe, salmart-dev, skjnldsv, sorbaugh and susnux and removed request for a team August 8, 2025 03:08
@skjnldsv skjnldsv merged commit 117d8de into master Aug 8, 2025
211 of 213 checks passed
@skjnldsv skjnldsv deleted the jtr-refactor-files-external-oauth1 branch August 8, 2025 06:22
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnux susnux susnux approved these changes

@skjnldsv skjnldsv skjnldsv approved these changes

@nfebe nfebe Awaiting requested review from nfebe nfebe is a code owner automatically assigned from nextcloud/server-frontend

@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend

@sorbaugh sorbaugh Awaiting requested review from sorbaugh sorbaugh is a code owner automatically assigned from nextcloud/server-backend

Assignees

No one assigned

Labels

3. to review Waiting for reviews feature: external storage ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@joshtrichards @susnux @skjnldsv