Skip to content

[Feature] Add user auth, database support to candis, change UI for the same - add signup, signin forms etc.#121

Merged
rupav merged 53 commits intoHelikarLab:developfrom
rupav:develop
Jul 14, 2018
Merged

[Feature] Add user auth, database support to candis, change UI for the same - add signup, signin forms etc.#121
rupav merged 53 commits intoHelikarLab:developfrom
rupav:develop

Conversation

@rupav
Copy link
Collaborator

@rupav rupav commented Jul 13, 2018

Explanation About What Code Achieves:
  • Explanation

  • Used Formik and Yup for making SignUp and SignIn forms.

  • User authentication using JWT tokens

  • database chosen is postgreSQL

  • envparse for parsing environment variables to set up the flask app config.

  • For database migrations - using flask-migrate

Tables Created:

  • Users - user details for auth, and relationship with other tables
  • Pipeline - for pipeline data, many-to-one relation with User table
  • CData - for storing cdata files/pandas dataframe, many-to-one relation with User table
  • PipelineRun - for storing data related to every run of a pipeline... like .gist files - This needs to be improved. Have a many-to-one relation with Pipeline table.
  • Response - for storing every endpoint's reponse - ID, status, data or error if any in JSON format
TODOs:
  • API endpoints needs to be converted using Swagger convention of ReST APIs
  • Improve PipelineRun Table.
  • Add a table to hold gist data.
  • to install and run postgres daemon through get-candis
  • Run postgres daemon through get-candis.

@rupav
Copy link
Collaborator Author

rupav commented Jul 14, 2018

Update
Connecting to database on travis build: env variables are passed using travis UI, refer settings of the travis.
Refer: https://docs.travis-ci.com/user/environment-variables/#Defining-Variables-in-Repository-Settings

because of https://docs.travis-ci.com/user/pull-requests/#Pull-Requests-and-Security-Restrictions, exposing env variables for travis build as non-sensitive testing data only.

@rupav
Copy link
Collaborator Author

rupav commented Jul 14, 2018

Merging this ✨

@rupav rupav merged commit 679a39b into HelikarLab:develop Jul 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant