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

Update Vuemit.js #40

Merged
merged 1 commit into from
Feb 3, 2021
Merged

Update Vuemit.js #40

merged 1 commit into from
Feb 3, 2021

Conversation

ctf0
Copy link
Contributor

@ctf0 ctf0 commented Jan 29, 2021

to fix issues with new wp 5 where we will get Uncaught TypeError: Vue is not a constructor,

note that this might break current setups, so better release a new major version.

to fix issues with new wp 5 where we will get `Uncaught TypeError: Vue is not a constructor`, to avoid break current setup, this should be on a new major version.
@gocanto
Copy link
Owner

gocanto commented Jan 30, 2021

Hi @ctf0, thank you for taking the time to fix this.

Question, how would this fix the issue?

@ctf0
Copy link
Contributor Author

ctf0 commented Jan 30, 2021

while using laravel mix which use wp5, i keep getting TypeError: Vue is not a constructor so to comply with using the esm build of vue we have to use .default.

@ctf0
Copy link
Contributor Author

ctf0 commented Feb 2, 2021

any news ?

@gocanto
Copy link
Owner

gocanto commented Feb 2, 2021

@ctf0 sorry, I have been really busy at work lately.

Let me pull these changes tomorrow, try them out and tag the release.

Thank you again!

@gocanto gocanto merged commit 508be66 into gocanto:master Feb 3, 2021
@gocanto
Copy link
Owner

gocanto commented Feb 3, 2021

@ctf0

Ok, it should be good to go. Thank you!

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