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

ingestion script #618

Merged
merged 3 commits into from
May 15, 2020
Merged

ingestion script #618

merged 3 commits into from
May 15, 2020

Conversation

jmensch1
Copy link
Contributor

@jmensch1 jmensch1 commented May 15, 2020

This converts the /ingest endpoint to a script that will be called as a cron-job on heroku.

The /ingest endpoint has been convenient for development, but we can't keep it around any more because it's a public endpoint that allows anyone in the world to delete and repopulate our production database.

Devs can populate their local DBs by logging into the backend container and running the script:

docker exec -it 311-backend /bin/bash
python ingest.py
  • Up to date with dev branch
  • Branch name follows guidelines
  • All PR Status checks are successful
  • Peer reviewed and approved

Any questions? See the getting started guide

@jmensch1 jmensch1 changed the title Back ingestion script ingestion script May 15, 2020
@jmensch1 jmensch1 requested review from adamkendis and sellnat77 May 15, 2020 17:02
Copy link
Member

@adamkendis adamkendis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call.

@jmensch1 jmensch1 merged commit 9f09860 into dev May 15, 2020
@jmensch1 jmensch1 deleted the BACK-IngestionScript branch May 15, 2020 18:18
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.

2 participants