-
Notifications
You must be signed in to change notification settings - Fork 10
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
Is v5 planned for this package? #77
Comments
+1 |
1 similar comment
+1 |
I'm was rewritten the alpha version for Adonis v5 TypeScript. You can try this |
Is there any update on this? Is this project abandoned, or is there another solution for v5? |
It is not abandoned. But I probably won't update it for Adonis 5 since I don't have any projects with v5. I wound not be able to support it. |
Would a PR be fine? I can work on porting the project to V5 and raise a PR if you're okay with it |
The problem isn't porting it to v5, but providing proper support. Since I'm not using v5, I won't be able to really test it, ensure good developer experience, and quickly react to a bug. I think it depends on how much work is required to support v5. If the interface remains largely the same and just a few changes are required for compatibility I'm fine with merging a PR. But if it requires basically a complete rewrite I don't think it wound be a good idea. I'm open for input. |
will check on this one. I used bumblebee for v4 and was amazing. |
Is v5 planned for this package or should I look for other solutions?
The text was updated successfully, but these errors were encountered: