search

fedora-copr / log-detective-website

Data collection page for Log Detective AI
11 stars 8 forks source link

docker: copy initial code to frontend container and set workdir #32

Closed FrostyX closed 9 months ago

FrostyX commented 9 months ago

Otherwise, we are basically running the npm install in root directory. See https://stackoverflow.com/a/65443098/3285282

jpodivin commented 9 months ago

Build is failing for me with:

log-detective-website-frontend-1  | [:app] Configuring build.
log-detective-website-frontend-1  | [:app] Compiling ...
log-detective-website-frontend-1  | [:app] Build failure:
log-detective-website-frontend-1  | The required namespace "react" is not available, it was required by "reagent/core.cljs".

The app doesn't seem to be running either, is it just me?

TomasTomecek commented 9 months ago

Interesting, I got the same actually:

frontend_1  | shadow-cljs - dependencies updated
frontend_1  | shadow-cljs - HTTP server available at http://localhost:3000
frontend_1  | shadow-cljs - server version: 2.26.2 running at http://localhost:9630
frontend_1  | shadow-cljs - nREPL server started on port 3333
frontend_1  | shadow-cljs - watching build :app
frontend_1  | [:app] Configuring build.
frontend_1  | [:app] Compiling ...
frontend_1  | [:app] Build failure:
frontend_1  | The required namespace "react" is not available, it was required by "reagent/core.cljs".
frontend_1  |

The frontend was served but without assets.

FrostyX commented 9 months ago

@jpodivin can you please try docker-compose build frontend --no-cache?

FrostyX commented 9 months ago

I am merging, this is IMHO a step in the right direction. We can iterate on the @jpodivin issue in a follow-up.