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

Dev/tiger #36

Merged
merged 21 commits into from Feb 24, 2021
Merged

Dev/tiger #36

merged 21 commits into from Feb 24, 2021

Conversation

lmeyerov
Copy link
Contributor

@lmeyerov lmeyerov commented Feb 24, 2021

Preparing merge of #30

Fixes:

  • flake8 lint warnings
  • missing plotly dependency
  • badges (unrelated)
  • propagate connection error message to user

Adds:

  • Docs: Setup flow for tg -> g-a-k
  • CI

@lmeyerov
Copy link
Contributor Author

lmeyerov commented Feb 24, 2021

@akash-kaul @zrougamed @jonherke

Continuing from slack, can you assist with fixing the Circle demo and ensure the other one works too? And should we merge first or fix first this before suggesting folks try it?

My current test (using the PR's fixes):

  1. src/envs/tigergraph.env:
TIGERGRAPH_HOST=https://zzz.i.tgcloud.io
TIGERGRAPH_USERNAME=tigergraph
TIGERGRAPH_PASSWORD=graphappkit
TIGERGRAPH_GRAPHNAME=AntiFraud
TIGERGRAPH_SECRET=zzz
  1. launch
cd src/docker
docker-compose build
docker-compose up
  1. Test

From there, http://localhost:8501 -> circle demo

@lmeyerov lmeyerov merged commit 5e17aeb into master Feb 24, 2021
@lmeyerov lmeyerov deleted the dev/tiger branch February 24, 2021 23:35
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.

None yet

1 participant