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

+ correctly expose methods option to recover compatibility to vue-class-component #87

Merged
merged 3 commits into from
Sep 21, 2023

Conversation

tall1on
Copy link
Contributor

@tall1on tall1on commented Sep 18, 2023

Hey, I noticed that the methods option doesn't work like the one from vue-class-component, so I added it. I also improved a bit of other code. Tests have all been passed. I also edited the methods test to make sure it keeps working like in vue-class-component.

@tall1on tall1on changed the title + correctly expose "methods"-option to recover compatibility to vue-class-component + correctly expose methods option to recover compatibility to vue-class-component Sep 18, 2023
@fb-sean
Copy link

fb-sean commented Sep 18, 2023

Really needed thanks for the PR!

src/component.ts Outdated Show resolved Hide resolved
src/option/emit.ts Outdated Show resolved Hide resolved
src/utils.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@ruojianll ruojianll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left some messages and hope you can improve it.

@tall1on
Copy link
Contributor Author

tall1on commented Sep 20, 2023

I guess i overthought it a bit 😅

I changed everything you asked, all tests passed.

@ruojianll ruojianll merged commit a41aa9b into facing-dev:master Sep 21, 2023
1 check passed
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.

3 participants