-
Notifications
You must be signed in to change notification settings - Fork 7
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
MVP Milestone #55
MVP Milestone #55
Conversation
This migration is only migrating the vue components. But later by bringing in some other changes, this will also make the application build for NixOS and other platforms. |
This needs to pull in changes from |
cva_test brings in specific changes from vue3 and temp_mvp is more for the nixos builder specific stuff |
Aiming for this PR to be merged by the end of this week. |
Current status:
|
@gideonairex if this PR can be merged within this week with working builds and ported-over components, then we can start working on drafting the UI. Please start writing comments here too instead of inside the issues. The issues are for general discussion about the task. Once the task is started, write comments into the PR instead. |
Current Status:
|
Are we merging this? |
I'm not able to answer the question about useStore vs useModule. Can you explain further? In the interests of time, make your best judgement and proceed. @gideonairex |
I think we can merge this on Monday, there were some issues with nix not being able download some npm packages, but I suspect that was just an issue with my connection. Once we resolve that issue and polykey is installable on our machines then we can go ahead and merge I think 👍 |
The nix and npm integration is being addressed in the TypeScript-Demo-Lib but we hit some issues with node2nix.
…On 1 November 2020 12:51:54 am AEDT, Robbie Cronin ***@***.***> wrote:
I think we can merge this on Monday, there were some issues with nix
not being able download some npm packages, but I suspect that was just
an issue with my connection. Once we resolve that issue and polykey is
installable on our machines then we can go ahead and merge I think :+1:
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
MatrixAI/Polykey#55 (comment)
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
The build is working now, but there are a few issues after installation has finished, namely:
|
Added tailwind so that we can easily create our composable components. Pushed sample usage on App.vue. On |
Changes:
Fixes #53
Fixes #54