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

Created migration mode in app #180

Merged
merged 1 commit into from
Nov 26, 2024
Merged

Created migration mode in app #180

merged 1 commit into from
Nov 26, 2024

Conversation

JoshD94
Copy link
Contributor

@JoshD94 JoshD94 commented Nov 24, 2024

Overview

Created a migration mode in app.py which is activated by setting the environment variable FLASK_MIGRATE to true

Changes Made

  • Wrapped imports and scrapers in condition, only run when FLASK_MIGRATE environment variable is set to false or is not set
  • Changed workflow file to set FLASK_MIGRATE to true before migrating, and then setting to false afterwards
  • Added FLASK_MIGRATE=false in Dockerfile to ensure no migrations occur during docker build

Test Coverage

Tested locally

Related PRs or Issues (delete if not applicable)

#169

@JoshD94 JoshD94 merged commit 1188f4e into master Nov 26, 2024
1 check passed
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