-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
🔧 PriceChart vue3 #6720
🔧 PriceChart vue3 #6720
Conversation
roiLeo
commented
Aug 15, 2023
- ref Reduce using nuxt-property-decorator and use composition API #4750
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
AI-Generated Summary: This pull request refactors the 'PriceChart.vue' component of the codebase, switching it from Vue 2 to Vue 3 Composition API structure. Vast changes include removal of the importing of ChainMixin, replacing the previously class-based structure of the component with the use of reactive 'ref' and 'watch' methods. Events are now emitted directly in the template. Furthermore, the 'mounted' lifecycle hook has been replaced with the 'onMounted' method, and a 'watch' declaration has been introduced to observe changes in 'priceData' and re-run 'priceChart'. Overall, the patch enhances code readability and leverages Vue 3's composition API for better state management. |
Reviewpad Report
|
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Code Climate has analyzed commit e7b8a5a and detected 0 issues on this pull request. View more on Code Climate. |