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

Directions for Windows / Docker Composer #597

Merged
merged 3 commits into from
Jul 16, 2024
Merged

Conversation

bryanthaboi
Copy link
Contributor

npm doesn't run it correctly (it seems like its working but only white screens) but docker composer through wsl2 (automatically installed these days) does the trick

@CLAassistant
Copy link

CLAassistant commented Jul 16, 2024

CLA assistant check
All committers have signed the CLA.

@jelveh
Copy link
Contributor

jelveh commented Jul 16, 2024

Thank you very much for this! We've historically had issues with Docker compose on Windows. We'll test and, hopefully, merge soon.

@jelveh jelveh requested a review from KernelDeimos July 16, 2024 16:26
@jelveh jelveh merged commit 9d8a468 into HeyPuter:main Jul 16, 2024
4 checks passed
@jelveh
Copy link
Contributor

jelveh commented Jul 16, 2024

Thank you very much for your contribution! Merged 🎉

@bryanthaboi
Copy link
Contributor Author

No problem! Made a video about it. https://youtu.be/gf7eBugidHc?si=mz6von52kJDRDTiw

@jelveh
Copy link
Contributor

jelveh commented Jul 17, 2024

@bryanthaboi Thank you so much for this!! Sorry Windows installation was broken. I'm going to fix it.

@bryanthaboi
Copy link
Contributor Author

Hey no problem i figured out what i needed to to try it out!

@KernelDeimos
Copy link
Contributor

@bryanthaboi following another mention of Windows support we've added a fix (d80f2fa) that should get npm install && npm start working under Windows as well.

@jelveh
Copy link
Contributor

jelveh commented Jul 19, 2024

@bryanthaboi we now have a "media" channel on Discord and I posted your video there as well: https://discord.com/channels/1071986474041495705/1263689151212032011/1263954371389952082

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.

4 participants