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.

Sign up for GitHub

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

Jump to bottom

Harvest db update #46

Merged
merged 6 commits into from
Mar 20, 2024
Merged

Harvest db update #46

merged 6 commits into from
Mar 20, 2024

Conversation

Jin-Sun-tts
Copy link
Contributor

@Jin-Sun-tts Jin-Sun-tts commented Mar 14, 2024 •
edited
Loading

Pull Request

Related to (GSA/data.gov#4647)

  • Switched to Flask-Alchemy
  • Reorganized the Flask app into separate directory.
  • Added Flask-Migrate for managing database DDL changes.
  • Added table indexes.

@Jin-Sun-tts Jin-Sun-tts marked this pull request as ready for review March 14, 2024 21:36
@Jin-Sun-tts Jin-Sun-tts requested a review from a team March 14, 2024 21:36
FuhuXia
FuhuXia reviewed Mar 15, 2024
View reviewed changes
app/models.py Outdated Show resolved Hide resolved
FuhuXia
FuhuXia reviewed Mar 15, 2024
View reviewed changes
app/models.py Outdated Show resolved Hide resolved
FuhuXia
FuhuXia reviewed Mar 15, 2024
View reviewed changes
app/models.py Outdated Show resolved Hide resolved
FuhuXia
FuhuXia reviewed Mar 15, 2024
View reviewed changes
app/models.py Outdated Show resolved Hide resolved
FuhuXia
FuhuXia reviewed Mar 15, 2024
View reviewed changes
app/models.py Outdated Show resolved Hide resolved
FuhuXia
FuhuXia reviewed Mar 15, 2024
View reviewed changes
requirements.txt Outdated Show resolved Hide resolved
@FuhuXia FuhuXia mentioned this pull request Mar 15, 2024
Merged
4 tasks
@Jin-Sun-tts Jin-Sun-tts requested a review from a team March 19, 2024 23:07
@Jin-Sun-tts Jin-Sun-tts requested a review from FuhuXia March 20, 2024 13:20
@Jin-Sun-tts Jin-Sun-tts merged commit f60156a into main Mar 20, 2024
4 checks passed
@Jin-Sun-tts Jin-Sun-tts deleted the harvest-db-update branch March 20, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FuhuXia FuhuXia FuhuXia approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Jin-Sun-tts @FuhuXia