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

[WIP] Add a script for porting data to the new models #48

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

Conversation

coditva
Copy link
Member

@coditva coditva commented Sep 18, 2017

To Do:

  • Add a framework for porting data according to a json object.
  • Make the script create tables if not existing.
  • Add transformation data.
  • Add cleaning techniques to data.
  • Create a module and import.

@coditva coditva self-assigned this Sep 18, 2017
@coditva
Copy link
Member Author

coditva commented Sep 18, 2017

Kind of addresses #46

@mukkachaitanya
Copy link
Member

@utkarshme Lodash provides excellent functionality to do these kinds of data structure manipulations. See if it suits your needs.

- Generalized to make porting easier by changing only the transform.js
file which is nothing but json.
@coditva
Copy link
Member Author

coditva commented Sep 26, 2017

Makes #50 redundant.

@coditva
Copy link
Member Author

coditva commented Sep 26, 2017

Read the documentation in the README to populate config.js

- bonafide
- day scholars
- day pass
- csa
- disco
- faculty
- facultyIncharge
- holiday
@SebastinSanty
Copy link
Member

Great work! From what I can comprehend, I feel that the user doesn't need to fiddle with the rules, which the script is already taking care of. I changed the other required variables, still, porting is not working for me.

@coditva
Copy link
Member Author

coditva commented Oct 4, 2017

Did you switch off the debug? Also, the script ports according to the models.sql file in src/data. Check if the ones in your database are same or not. Create a new DB and try?

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