Skip to content
This repository was archived by the owner on Jul 20, 2019. It is now read-only.

Adds a separate UI based on create-react-app#28

Open
jehrhardt wants to merge 39 commits intotechnology-ebay-de:masterfrom
jehrhardt:separate-ui
Open

Adds a separate UI based on create-react-app#28
jehrhardt wants to merge 39 commits intotechnology-ebay-de:masterfrom
jehrhardt:separate-ui

Conversation

@jehrhardt
Copy link
Contributor

No description provided.

infotexture and others added 30 commits December 7, 2018 13:48
The project provides a docker-compose based dev and build environment.
This will unify environments across machines.
This also fixes some reported vulnerabilities.
React will be used to build the UI.
The sketch pad is the root aggregate in the coding bounded context. It
will contain all the coding related stuff.
Phoenix generates some defaults. These are not required and must be
removed.

Also React.js is not needed everywhere 😉.
The generated tests fail after changing the code 🤨.
In the future the UI will become a separate subproject. Thus the API
must become a separate subproject as well.

Also the previous UI experiment must be renamed, so the new one can be
named ui.
This is an early prototype for the UI.
At the moment docker-compose is easiest way to run ui, api and Postgres
during development. It reduces the amount of tools, that developers
need to install.
The commands have changed and must be updated for CI.
We will use Docker to build the software, but we will not use it to run
tests. Docker is just slowing the Travis CI build down.
Ehrhardt, Jan added 9 commits May 26, 2019 13:29
The instructions are now for a more local setup with a reduced usage of
Docker. This makes working with the code easier.

Later we will probably use VS Code's upcoming remote feature with
Docker.
This will make the tooling work out of the box.
The documentation is provided in the project root.
Docker will be used to build and package the ui and api. This is not finished
yet, but at least finished so far it is ready.
The controller has been generated by Phoenix is not needed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants