Skip to content
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

refactor: convert Vue components from Class API to Options API #2352

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

davinov
Copy link
Member

@davinov davinov commented Feb 28, 2025

Summary

  • Converted all Vue components from Class API (with vue-property-decorator) to Options API with defineComponent
  • Removed vue-class-component and vue-property-decorator dependencies
  • Updated component patterns to use proper TypeScript typing with PropType
  • Fixed test utilities to support Options API components
  • Added migration documentation in CLAUDE.md

Test plan

  • All unit tests now pass with the new component structure
  • No functionality changes, only refactoring of the component implementation style
  • Verified proper typing of all components

🤖 Generated with Claude Code

- Converted DataViewer component to Options API
- Removed vue-class-component and vue-property-decorator dependencies
- Updated test utilities to support Options API
- Fixed syntax errors in ConditionsEditor
- Updated migration documentation to reflect completion

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <[email protected]>
- Remove vue-class-component and vue-property-decorator dependencies
- Convert all components to use defineComponent for better TypeScript support
- Preserve component functionality while simplifying implementation
- Update test utilities to work with Options API

This refactor improves maintainability, reduces dependencies, and
aligns with Vue's recommended practices.
@davinov davinov force-pushed the refactor/class-to-options-api-migration branch from 0ab3f8b to 69520ce Compare February 28, 2025 14:28
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.

1 participant