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

Project/local units #2113

Merged
merged 15 commits into from
May 8, 2024
Merged

Project/local units #2113

merged 15 commits into from
May 8, 2024

Conversation

k9845
Copy link
Contributor

@k9845 k9845 commented Apr 18, 2024

Addresses

Please run the following commands

from local_units.models import LocalUnit, HealthData
LocalUnit.objects.all().delete()
HealthData.objects.all().delete()
docker-compose exec  serve bash ./manage.py loaddata health-affiliation-functionality.json
docker-compose exec serve bash ./manage.py loaddata local-unit-level.json

Data Import Script to run

docker-compose exec serve bash ./manage.py import_health_data "Health Services Mapping Table B.csv"
docker-compose exec serve bash ./manage.py import_local_units_health A.csv 

Checklist

Things that should succeed before merging.

  • Updated/ran unit tests
  • Updated CHANGELOG.md

Release

If there is a version update, make sure to tag the repository with the latest version.

@k9845 k9845 force-pushed the project/local-units branch from 8badf13 to 7ff81e5 Compare April 22, 2024 05:36
@szabozoltan69
Copy link
Contributor

szabozoltan69 commented Apr 22, 2024

@k9845 Please do not replace health_id with DATA_SOURCE_ID.
The request (from you) was to have only one field referring from TableA to TableB, and it's a foreign key, health_id.
(And please rebase with develop.)

@k9845 k9845 force-pushed the project/local-units branch 5 times, most recently from 202b037 to 422465e Compare April 26, 2024 08:36
@samshara samshara requested a review from thenav56 May 2, 2024 11:46
@k9845 k9845 force-pushed the project/local-units branch 11 times, most recently from 4de1247 to 79a8869 Compare May 7, 2024 06:43
@k9845 k9845 force-pushed the project/local-units branch from ff20376 to 12f7a37 Compare May 8, 2024 04:40
@k9845 k9845 marked this pull request as ready for review May 8, 2024 05:03
@k9845 k9845 force-pushed the project/local-units branch from 77c462d to f7da3b5 Compare May 8, 2024 05:35
@frozenhelium
Copy link
Member

@szabozoltan69 Let's review, merge and deploy this one.

@szabozoltan69 szabozoltan69 merged commit 8fc74ab into develop May 8, 2024
1 check passed
@szabozoltan69 szabozoltan69 deleted the project/local-units branch May 8, 2024 11:00
@szabozoltan69
Copy link
Contributor

Deployed. Load scripts also run.
Results of loading table A and B.

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