Adds Dockerfile

[sladyn98]

Added simple dockerfile for frontend and backend to be able to run the project easily.

[sladyn98]

I will add new lines at the end of the files if required :)

[oleg-nenashev]

Issue found: Use COPY instead of ADD for files and folders

[kwhetstone]

For testing these files together, do you want to put together a docker-compose? Adding directions to the README will help too.

[sladyn98]

@kwhetstone Yeah my initial aim was to get a docker compose file but the front end service throws a ton of errors when run using a docker compose, I will instead add instructions to the readme just so that we can run these separately.

[LinuxSuRen]

I will add new lines at the end of the files if required :)

You can find some reasons for it from here.

[sladyn98]

Commands for running the front end are, these should be run from the front-end directory
docker build -t cds_react:dev .
docker run \ -it \ --rm \ -v ${PWD}:/app \ -v /app/node_modules \ -p 3001:3000 \ -e CHOKIDAR_USEPOLLING=true \ cds_react:dev

Could you just try this on your local machine and see if it running because it is throwing me a bunch of errors @LinuxSuRen

[LinuxSuRen]

I can run it by the following command:
docker run -it --rm -v ${PWD}:/app -p 3001:3000 -e CHOKIDAR_USEPOLLING=true cds_react:dev

But I'm not sure it can really work.

[sladyn98]

@LinuxSuRen the command is failing on my machine I mean the docker container is still not running

[sladyn98]

I generated the package-lock.json again @kwhetstone in order to make this run. I also removed the eslint scripts. I guess we can just add them later

[kwhetstone]

I think the big thing for this was to use eslint 6 instead of 7 since it was causing issues. I know they're different, and eslint 6 will be supported for a bit here since there's so much built on it.

[sladyn98]

This file now only contains the dockerfile update, and the eslint version has been fixed in another PR here it is #37 . So kindly merge that before this one.

[kwhetstone]

Let's get this in.

[sladyn98]

@kwhetstone Roger that 🚀