Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Oct 28, 2025

Summary

  • migrate all files to Typescript
  • migrate vue to script-setup
  • migrate Vuex to Pinia
  • migrate to Vue 3

best to view commit by commit.

Checklist

@susnux susnux added this to the Nextcloud 33 milestone Oct 28, 2025
@susnux susnux self-assigned this Oct 28, 2025
@susnux susnux added 3. to review Waiting for reviews technical debt ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Oct 28, 2025
@susnux susnux requested review from artonge, leftybournes, sorbaugh, szaimen and yemkareems and removed request for a team October 28, 2025 17:47
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/password-confirmation": "^6.0.1",
"@nextcloud/paths": "^2.2.1",
"@nextcloud/vue": "^9.0.1",
"pinia": "^3.0.3",
Copy link
Member

Choose a reason for hiding this comment

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

Whaaaaat, how is this an addition?
We use Pinia on files, the dep should already be here 🤔

Copy link
Member

Choose a reason for hiding this comment

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

AH yes, I forgot we moved the pkg into build

Copy link
Contributor Author

Choose a reason for hiding this comment

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

its the first Vue 3 app that uses pinia :)

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
…pescript

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the refactor/twofactor-vue3 branch from 9048741 to 9008175 Compare November 1, 2025 11:40
@susnux susnux merged commit f67d90a into master Nov 1, 2025
218 of 226 checks passed
@susnux susnux deleted the refactor/twofactor-vue3 branch November 1, 2025 12:41
@github-project-automation github-project-automation bot moved this to 🏗️ In progress in 📁 Files team Jan 5, 2026
@susnux susnux moved this from 🏗️ In progress to ☑️ Done in 📁 Files team Jan 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) technical debt ❄️ 2026-Winter

Projects

Status: ☑️ Done

Development

Successfully merging this pull request may close these issues.

4 participants