Skip to content

Update development notes - #75

Open
Ghesselink wants to merge 1 commit into
AECgeeks:developmentfrom
Ghesselink:update_local_dev_notes
Open

Update development notes#75
Ghesselink wants to merge 1 commit into
AECgeeks:developmentfrom
Ghesselink:update_local_dev_notes

Conversation

@Ghesselink

Copy link
Copy Markdown
Contributor

Relatively simple method to run React frontend and Flask backend on a local machine without using docker.

Comment thread README.md
Update and install submodules

~~~
submodule init && submodule update

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

git submodule update --init --recursive

Comment thread README.md
Set environment to 'development'

~~~
export environment=development

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

export is for linux/wsl, on windows use set

Comment thread README.md

Setup local environment with the following packages
~~~
pip install lark-parser==0.12.0 numpy ifcopenshell flask flask-cors numpy gunicorn rq redis SQLAlchemy psycopg2 psycopg2cffi sqlalchemy-utils Flask-BasicAuth flasgger requests XlsxWriter requests_oauthlib Authlib requests argparse orjson --upgrade && conda install flake8

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

psycopg2 psycopg2cffi are not needed, why not use pip to install flake8 as well?

Comment thread README.md

Depending on the OS, change ui/package.json. For instance to 'bash run_debug.sh' in case of a linux system.
~~~
"api": "cd ../application && run_no_debug.bat"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider adding api-windows and api-linux or sth instead of just api to package.json so that no local change need to be made.

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.

2 participants