Skip to content

refactor(forms): Migrate Forms.vue to Vue 3 Composition API #2762

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.

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nfebe
Copy link
Contributor

@nfebe nfebe commented Jun 10, 2025

refactor(forms): Migrate Forms.vue to Vue 3 Composition API

  • Converted component to use Vue 3's Composition API with setup()
  • Removed duplicate function declarations and fixed syntax issues
  • Updated lifecycle hooks to use onMounted/onUnmounted
  • Fixed reactivity using ref and computed
  • Cleaned up event handling and component structure
  • Ensured compatibility with Vue 3 and Nextcloud ecosystem

@nfebe nfebe force-pushed the refactor/migrate-2-vue3 branch 3 times, most recently from 50d735f to 1cd6f58 Compare June 10, 2025 19:29
- Converted component to use Vue 3's Composition API with setup()
- Removed duplicate function declarations and fixed syntax issues
- Updated lifecycle hooks to use onMounted/onUnmounted
- Fixed reactivity using ref and computed
- Cleaned up event handling and component structure
- Ensured compatibility with Vue 3 and Nextcloud ecosystem
@nfebe nfebe force-pushed the refactor/migrate-2-vue3 branch from 1cd6f58 to c8fe715 Compare June 10, 2025 19:30
@Chartman123
Copy link
Collaborator

Hi @nfebe

really nice, thanks for your efforts! 🎉

Before we merge this, I'd like to merge the other open PRs #2737 and #2748.

@nfebe
Copy link
Contributor Author

nfebe commented Jun 11, 2025

Hi @Chartman123 No worries at all, this is still a very rough draft.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants