Skip to content
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

Corrections_NormaW_LeonH_PullReq_001 #43

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

leonhummernbrum
Copy link

No description provided.

leonhummernbrum and others added 9 commits August 11, 2022 16:02
@leonhummernbrum leonhummernbrum changed the title Corrections norma w leon h Corrections_NormaW_LeonH_PullReq_001 Sep 6, 2022
@@ -23,7 +23,7 @@ Execute this commands to install Zendro:
$ git clone https://github.com/Zendro-dev/zendro.git
$ cd zendro
$ npm install
$ npm link
$ npm link #sudo npm link?
Copy link
Member

Choose a reason for hiding this comment

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

remove comment.

@@ -88,7 +88,7 @@ $ docker ps

You can check docker logs by:
```
$ docker logs -f <container name>
$ docker logs -f <container name> #for us the container id worked, the name did not
Copy link
Member

Choose a reason for hiding this comment

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

remove comment


* **Without docker setup:** ./graphql-server/config/data_models_storage_config.json
* **With docker setup:** ./config/data_models_storage_config.json
* **Without docker setup:** ./graphql-server/config/data_models_storage_config.json (specify data model here)
Copy link
Member

Choose a reason for hiding this comment

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

data model -> data models

NormaW and others added 6 commits September 8, 2022 08:11
line 123 Keycloak: 
line 124 http://localhost:8081/auth

without /auth the link does not work, in our cases
added empty link for "How to uppload data using the SPA"
added yarn as required software
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.

3 participants