Skip to content

refactor(files_external): migrate to script-setup and Vue 3 #57237

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
Open
susnux wants to merge 1 commit into
base: master
Choose a base branch
from
Open

refactor(files_external): migrate to script-setup and Vue 3 #57237

susnux wants to merge 1 commit into from

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Dec 23, 2025 •
edited
Loading

Summary

Migrate to script-setup where possible, add some missing doc strings and use Vue 3 for the few components.

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Dec 23, 2025
@susnux susnux added the 3. to review Waiting for reviews label Dec 23, 2025
@susnux susnux requested review from a team and skjnldsv as code owners December 23, 2025 19:10
@susnux susnux requested a review from a team as a code owner December 23, 2025 19:10
@susnux susnux added technical debt ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Dec 23, 2025
@susnux susnux requested review from come-nc, nfebe, provokateurin and sorbaugh and removed request for a team December 23, 2025 19:10
@susnux susnux mentioned this pull request Dec 23, 2025
Open
31 tasks
@susnux susnux force-pushed the refactor/files-external-vue3-ts branch from 11227a3 to 25a0d78 Compare December 23, 2025 19:13
@come-nc come-nc removed their request for review January 6, 2026 13:01
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 7, 2026
Merged
CarlSchwan
CarlSchwan approved these changes Jan 7, 2026
View reviewed changes
Copy link
Member

@CarlSchwan CarlSchwan left a comment

Choose a reason for hiding this comment

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

Nice :)

@susnux susnux requested a review from szaimen January 7, 2026 21:26
szaimen
szaimen approved these changes Jan 7, 2026
View reviewed changes
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

@nextcloud-bot nextcloud-bot mentioned this pull request Jan 9, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CarlSchwan CarlSchwan CarlSchwan approved these changes

@szaimen szaimen szaimen approved these changes

@skjnldsv skjnldsv Awaiting requested review from skjnldsv skjnldsv is a code owner

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

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

@provokateurin provokateurin Awaiting requested review from provokateurin provokateurin 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) technical debt

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

4 participants

@susnux @CarlSchwan @szaimen