-
Notifications
You must be signed in to change notification settings - Fork 75
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
refactor: migrate Vue 2 to 3 #57
base: master
Are you sure you want to change the base?
Conversation
Migrate Vue 2 to 3 CaliDog#52
Hey @ninoseki, this is great! Huge thanks for putting in the legwork to get this ported over. There's a few auto-formatting things that are sorta wonky, but definitely not a huge deal. You rock! 🙏 🙇 |
2 quick things:
|
Sorry, it's my bad. 🙇
I agree with you that including built assets a straightforward solution to skip the frontend building processes even if it is no recommended. |
Docker's multi-stage builds is a right solution to build the frontend in a single Dockerfile.
I will add commits to:
if it makes sense with you. |
@Fitblip What do you think which one is better?
|
Migrate Vue 2 to 3 #52