Solves issue#37 candis not running on docker

[promehul]

Explanation About What Code Achieves:

• It maps the redis layer which is running on the container instead of trying to find redis on the localhost.
• After this candis is running normally on docker. I am able to create pipelines and do stuff.

Screenshots/GIFs:

Steps To Test:

-Run it.

TODOs:

• None

[promehul]

Tests on this PR failed with error messages "no environment variables set". I am not able to get what I did wrong !!

[promehul]

Then merge it if you find it correct!!

[achillesrasquinha]

I don't think this is a good idea of providing the service name as a default. It should be localhost.

[achillesrasquinha]

Your service name should be overriden by environment variables passed into your docker service.

[achillesrasquinha]

Closing this.

[promehul]

@achillesrasquinha please check now if its what you asked for.

[rupav]

@promehul are you still facing that refresh problem - pipeline not saving in db on refreshing?

[promehul]

Its getting saved in db but it is not getting displayed ON REFRESH !! @rupav

[rupav]

just created a pipeline named 'pilot', logged out, logged in back pipeline is still present. Refreshed, then logged in, and again found the pipeline correctly.

[rupav]

Please check again, and let us know. Install a screen recorder (like 'peek'), to record the flow of the app, and thus the error you are facing, post the gif here when you are done.

[promehul]

@rupav are you running it on docker or using get_candis script? I followed the same steps that you followed .

[rupav]

One error found in the flow so far, forgot password functionality. That's a seperate issue, because of Gmail auth of our mail server. Except this, try finding other issues in docker setup @promehul . I am running it on docker @promehul using your changes.

[promehul]

If it is running all fine, then please merge my PR.

[promehul]

@rupav I did this

[rupav]

You have to open the existing file, not creating new. How can it create new with the same name, rather try opening the existing file. All works fine then.

[rupav]

This is the correct behavior of the app, if pipe with same name exists, you cant create one, and thus the warning.

[rupav]

try clicking on 'open' instead of 'new' after refreshing the browser, once you have created 'qwerty' pipe.

[promehul]

Thanks @rupav

[promehul]

This has been open for 9 days. Any follow up? @rupav

[rupav]

Did you @promehul try running the pipeline on docker? Are you able to populate the 'file' (by creating new file) with CEL files? If so, please explain how so. Refer Readme GIF, for checking how to run the pipeline.