Skip to content

refactor(encryption): migrate to Vue 3 and Typescript and script setup #57515

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 2 commits into
base: master
Choose a base branch
from
Open

refactor(encryption): migrate to Vue 3 and Typescript and script setup #57515

susnux wants to merge 2 commits into from

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jan 13, 2026 •
edited
Loading

Summary

  • migrated settings to Vue 3
  • Adjusted backend:
    • use strong types to get a http error 400 instead of 500 if wrong parameters are sent
    • properly handle exceptions thrown (prevent http error 503 when it should be a http 500)

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Jan 13, 2026
@susnux susnux requested review from a team as code owners January 13, 2026 13:51
@susnux susnux added 3. to review Waiting for reviews feature: encryption (server-side) ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Jan 13, 2026
@susnux susnux requested review from CarlSchwan, artonge, come-nc, icewind1991, leftybournes, nfebe, provokateurin, sorbaugh and szaimen and removed request for a team and sorbaugh January 13, 2026 13:51
@susnux susnux mentioned this pull request Jan 13, 2026
Open
22 tasks
szaimen
szaimen approved these changes Jan 13, 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.

🐘

@susnux susnux force-pushed the refactor/encryption-vue3 branch 2 times, most recently from 912fb9b to 98c9c3c Compare January 13, 2026 14:19
@susnux susnux mentioned this pull request Jan 13, 2026
Open
32 tasks
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 12, 2026
Merged
@susnux susnux force-pushed the refactor/encryption-vue3 branch from 98c9c3c to f309c3a Compare January 13, 2026 17:40
@susnux
Copy link
Contributor Author

susnux commented Jan 13, 2026

/compile

@nextcloud-command
chore(assets): Recompile assets
60f67d6 
Signed-off-by: nextcloud-command <[email protected]>
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

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

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

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

@artonge artonge Awaiting requested review from artonge

@provokateurin provokateurin Awaiting requested review from provokateurin

@come-nc come-nc Awaiting requested review from come-nc

Assignees

No one assigned

Labels

3. to review Waiting for reviews feature: encryption (server-side) ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

5 participants

@susnux @CarlSchwan @szaimen @nextcloud-command