-
Notifications
You must be signed in to change notification settings - Fork 33
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
BAH -1981 | Validate and update script to delete all patient data from Bahmni #25
base: master
Are you sure you want to change the base?
Conversation
To summarize, it seems you have:
Is my understanding correct? Are there any other changes? Is this tested for v0.93 or Bahmni LITE? Note: Not everyone uses docker with Bahmni. We may need an option when people run this command to say.. are you running this for docker, or running this on plan OS (like if people want to run this script on Centos/Vagrant Box). Maybe a switch that you pass to function saying |
In Bahmni LITE, people also use Crater instead of OpenERP/Odoo. We will also need a script to drop all CRATER data in future. FYI. |
@gsluthra, Yes i have moved delete patients data function into one script for the OpenELIS, OpenERP, OpenMRS , and i have tested for bahmni lite in docker, i will add that flag |
The previous script used environment variables to set DB connection creds. In the modified scripts now the default credentials are hardcoded. I think we should continue to provide the ability to use env variables to execute commands. Maybe set variables in start of the script, and re-use them at bottom, so people can edit them before running the script? |
Ok, I will change it |
No description provided.