-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add Docker support #172
Add Docker support #172
Conversation
bb2c975
to
29d2633
Compare
09c01ad
to
f0fb789
Compare
…/digital-fuesim-manv into feature/117-dockerization
Regarding production flags: Backend
Frontend
Shared
RootNo MethodI looked through all |
In my opinion, we should handle deployment in another issue/PR as it is not 100% tied to the dockerization. What are your thoughts on that? |
Immer is also used in the frontend (because it is in shared). I found this workaround to make it run in production in the frontend too: |
Sure. |
Ah, right. I completely forgot the frontend is no node app... I think this approach should be possible without issues. What would be the best place to add this snippet? There is not really an |
|
I have no clue why this fails. VSCode tells me |
We don't have the node typings in the frontend, but vscode doesn't get that. I would just use |
Ah, I see. Fixed it by adding some |
@Dassderdie Your issue should be fixed and this PR should be ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still get the same error with cannot read env of undefined
(probably because of docker-compose reusing some not invalidated cache?).
I set a breakpoint on main.js and manually added window.process = {}
and it worked. So if it works for you it's good to go.
You can do something like |
worked 👍 |
Closes #117
Supersedes #144
What did we change in this PR?
docker-compose up -d
in the root directory. SeeREADME.md
for more details.start:once
for thebackend
.From #117: