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

Added Vue 3 Composition API [1 of 2] #13

Merged
merged 4 commits into from
Nov 1, 2023
Merged

Added Vue 3 Composition API [1 of 2] #13

merged 4 commits into from
Nov 1, 2023

Conversation

delano
Copy link
Contributor

@delano delano commented Nov 1, 2023

This pull request introduces the use of the script setup syntax in our Vue 3 application, specifically within the App.vue file. The changes made in the past 4 commits are as follows:

Commit 1: Initialized the use of script setup in App.vue. This new syntax allows us to write our Composition API code without needing to define a setup() function, making our component code more concise.

Commit 2: Refactored existing reactive data properties to use the ref() function from Vue 3. This change was necessary to comply with the Composition API's way of handling reactivity.

Commit 3: Moved computed properties and methods into the script setup block. This change further aligns our code with the Composition API style and takes advantage of Vue 3's improved reactivity system.

Commit 4: Finalized the transition to script setup by ensuring all component options (such as props, components, etc.) are correctly defined and accessible within the script setup context.

These changes collectively improve the readability and maintainability of our Vue.js codebase by fully utilizing the advantages of Vue 3's Composition API.

Signed-off-by: GitHub Copilot

@delano delano self-assigned this Nov 1, 2023
@delano delano changed the title Added Vue 3 Composition API Added Vue 3 Composition API [1 of 2] Nov 1, 2023
@delano delano merged commit cccbc19 into main Nov 1, 2023
4 checks passed
@delano delano deleted the wip/oct30-django-vite branch November 1, 2023 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant