search

TooAngel / worlddriven

Automatic and well-defined pull request merged based on contribution-based weighted voting
GNU Affero General Public License v3.0
17 stars 14 forks source link

Migrate database backend to PostgreSQL #218

Closed TooAngel closed 3 years ago

TooAngel commented 3 years ago

The MongoDB addon used on heroku will shutdown soonish. Instead of MongoDB PostgreSQL is now used. Fixes #192

Flask-SQLAlchemy as ORM. Flask-Migrate handling the migration

On app startup the existing data is automatically migrated.

The github access keys are now stored encrypted, the encryption key is provided as ENCRYPTION_KEY as environment variable. Fixes #197

Fixes #49 - The session are and will be stored in the database

worlddrivenbot commented 3 years ago

This pull request will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). The start date is based on the latest Commit date / Pull Request created date / (force) Push date. The time to merge is 5 days plus 5 days for each commit. Check the worlddriven status check or the dashboard for actual stats.

To speed up or delay the merge review the pull request:

  1. Files changed
  2. Review changes
worlddrivenbot commented 3 years ago

The branch of this pull request was updated so the auto-merge time has been reset.

It will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). Check the worlddriven status check or the dashboard for actual stats.

worlddrivenbot commented 3 years ago

The branch of this pull request was updated so the auto-merge time has been reset.

It will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). Check the worlddriven status check or the dashboard for actual stats.

worlddrivenbot commented 3 years ago

The branch of this pull request was updated so the auto-merge time has been reset.

It will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). Check the worlddriven status check or the dashboard for actual stats.

worlddrivenbot commented 3 years ago

The branch of this pull request was updated so the auto-merge time has been reset.

It will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). Check the worlddriven status check or the dashboard for actual stats.

worlddrivenbot commented 3 years ago

The branch of this pull request was updated so the auto-merge time has been reset.

It will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). Check the worlddriven status check or the dashboard for actual stats.

worlddrivenbot commented 3 years ago

The branch of this pull request was updated so the auto-merge time has been reset.

It will be automatically merged by worlddriven in 1 day(s) and 4 hour(s). Check the worlddriven status check or the dashboard for actual stats.

worlddrivenbot commented 3 years ago

Thank you for the review. This pull request will be automatically merged by worlddriven in -1 day(s) and 22 hour(s). Current votes: 191/209.

Check the worlddriven status checks or the dashboard for actual stats.

TooAngel commented 3 years ago

This pull request was merged by worlddriven.

TooAngel commented 3 years ago

Never sure if I should approve things like this. I've looked through it and nothing seems wrong to me but I don't have enough experience with it to feel entirely confident that it is correct. I'm taking the view that it will be merged with or without me so I might as well look over it to see if anything jumps out at me as wrong and if not reduce the merge time.

Perfect. This way at least stupid typos could be found.

I think, not reviewing and clicking the button shouldn't be done. A proper review would be excellent, but at least a second pair of eyes is good.