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

Add alembic #804

Merged
merged 70 commits into from
Nov 2, 2020
Merged

Add alembic #804

merged 70 commits into from
Nov 2, 2020

Conversation

mattyweb
Copy link
Collaborator

@mattyweb mattyweb commented Sep 9, 2020

(Branched from 'api-new-version')

First version of alembic.

Adds tables for:

  • councils
  • regions
  • request_types

Seeds them with values.

Add materialized view for:

  • service_requests

Will also seed requests table in database if TESTING env is set

To see sample database run:

TESTING=True alembic upgrade head

Matthew Webster and others added 30 commits August 27, 2020 13:04
First set of API refactor changes
8f7b6ab
added pins route and Gzip
d4c65fa
first draft of legacy endpoints
63cc3b7
fixing lint issues from flake8
8886ca7
added support for deploying new API in Docker
35628ff
new date parsing and config changes
bd4ef1e
more docker and config changes plus README
cc995e0
default LEGACY_MODE to True
58a340a
DB seed needs database URL
d1b5daf
YAML fix
e53dbab
compose networking
f32f628
set test default config
11b9441
updated missing .env.example settings
aef7f67
@jmensch1
postman tests accept either 400 or 422 for bad input
a4cb368
@jmensch1
comparison set validation
5b916f8
model models and added simple entity cache
a1f94c7
added cache status check
986ba3e
fixing flake8
6ed9777
disabling cache build
1dcf5f5
first version of alembic
3911501
added testing
d7bc8f3
updating ENV and requirements
e4b4fc7
turned testing off
69fec6c
test tweaks
5316119
dockerignore changes
d8379dd
adding cwd to pythonpath
fa68515
fixed db env settings
05ac6ab
setting db to test in env
0c473fa
don't add _test to db name if there
c6e7a79
more reliable tests
b8795b9
jmensch1 and others added 26 commits September 13, 2020 09:35
@jmensch1
more postman logging
a9c970a
@jmensch1
commented build_cache
a66ea53
@jmensch1
linting
a066a0c
@jmensch1
logging 500 errors for all positive postman tests
1f0df0f
revert to 2 databases
f40a7cd
Merge branch 'add-alembic' of https://github.com/hackforla/311-data i…
60702ed 
…nto add-alembic
testing to false
7c05209
remove views from db_seed
2205bc7
removed db reset
d846040
added data file
9c09fd5
removing TESTING env
070d75e
removing from api.env
65a5335
fixing flake8
fa7de52
error improvement
8dc6a0b
1 more change
711b39f
undo docker compose change
60589b0
test database logging
d649c45
keep test db and new db fixture
a89155b
seed file and better alembic logging
a2698cf
adding env for seed
fd0e99c
fix env
af2135d
weird cluster fix
d53f499
LAST_WEEK to YEAR_TO_DATE in cypress
a59fe2f
type selector change
87ffbd0
direct request type all
ebd5b1b
handling 10K+
6829636
@mattyweb
Copy link
Collaborator Author

finally got everything working...
major changes include not deleting the database after the unit tests and setting up the database unit tests a little differently
had to fix some cypress issues with some selectors not working and a formatting issue on the clusters

@jmensch1 jmensch1 merged commit 46cd64a into dev Nov 2, 2020
@jmensch1 jmensch1 deleted the add-alembic branch November 2, 2020 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@mattyweb @jmensch1