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

Support for Vue 3.x #63

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

Support for Vue 3.x #63

wants to merge 6 commits into from

Conversation

nazar1ua
Copy link

I refactored this library on Vite for Vue 3.x, I tested it locally, but you can test it as well. The API has not changed.

@masterkain
Copy link

merge and release? 🙏

@donovanroubos
Copy link
Member

Hi @nazar1ua!

Thanks a lot for contributing and bumping the codebase to a Vue3 compatible dependency 🙏. We'll have a look upcoming days and test the solution and eventually merge and release a new version.

We're also exploring a new kind of architecture. More on that later.

Cheers!

Cc @masterkain

@SpiritusDeos
Copy link

SpiritusDeos commented Mar 22, 2023

Hi @donovanroubos @Davidkooi,

Any news regarding this PR, is this branch have a chance to be merged ?
It will be really useful for people using Vue 3 / Nuxt 3 🙏🏻

@donovanroubos
Copy link
Member

Hi @donovanroubos @Davidkooi,

Any news regarding this PR, is this branch have a chance to be merged ? It will be really useful for people using Vue 3 / Nuxt 3 🙏🏻

Hi @KennyyS!

Thanks for the kind reminder. We're currently busy with the new approach (in our spare time), the library that we'll be supporting is React because we'll start with a React POC. More on that later.

@camilotobar
Copy link

Hi everyone! Any updates on this merge? Is very important for TS and Vue3 projects.

@AntonLindell
Copy link

Would love to see this merge!

@nazar1ua
Copy link
Author

As a workaround, you can try installing flagpack-core and copying Flag.vue from my PR to your project

@miguilimzero
Copy link

It would be really nice to have this package up to date!

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.

7 participants