Skip to content
This repository has been archived by the owner on Apr 20, 2020. It is now read-only.

Refactor backend for TCN compliance #12

Open
wants to merge 49 commits into
base: master
Choose a base branch
from
Open

Refactor backend for TCN compliance #12

wants to merge 49 commits into from

Conversation

rettetdemdativ
Copy link
Contributor

  • Merge upload and download APIs
  • Add required functions instead of seemingly random ones
  • Remove unnecessary clutter

assert-not-singularity and others added 30 commits April 2, 2020 22:07
@assert-not-singularity
Add Dockerfile
c9027fa
Create functions for creating blueprints with database object
f28f949
Use pymondo instead of flask_pymongo
2dc0d29 
flask_pymongo was causing problems because it automatically connects to mongoDB on localhost and ignored any URI passed to it. Also it features an older version of pymongo, which is totally unnecessary.
Create routes with new blueprint creation functions and pass database…
44f6b7c 
… to them
Remove flask-pymongo as dependency
e9bf2af
Remove flask-pymongo as dependency and add pymongo
de0ec97
Bind flask to 0.0.0.0 as host in Docker container
8d240b9
Add docker-compose file
59f56b6
Adapt tests to new structure
446fe60
Use environment variables in Docker files
4304090
Merge branch docker into merge-download
d4b97bc
Merge branch docker into merge-download
b72437c
Change directory structure
4e019eb
Remove app directory
03100a6
Rename persistence and model packages
9c6d3cc
Format with black
41e9704
Move test helper functions out of db module
d757314
Rename main.py to app.py
13df4ce
Add get_cases function
98e82c5
Add handler to get cases
f3ba791
Move random_time_in_the_past to db module:
00a9674
@rettetdemdativ
Format with black
0e5eaea
@rettetdemdativ
Fix typing errors
ca82bf8
@rettetdemdativ
Format with black
68655b5
@rettetdemdativ
Install mongodb in actions pipeline
15cd5df
Merge pull request #11 from ito-org/actions-test
64eafad 
Install mongodb in actions pipeline
@rettetdemdativ
Try testing with localhost mongodb
d80baff
@rettetdemdativ
Use GitHub secret
3cf5303
@rettetdemdativ
Remove unused models and routes
fd4423e
@rettetdemdativ
Remove unused models and routes
f209855
@rettetdemdativ rettetdemdativ changed the title Refactor backend for TCN compliance [WIP] Refactor backend for TCN compliance Apr 10, 2020
@rettetdemdativ rettetdemdativ self-assigned this Apr 10, 2020
@rettetdemdativ rettetdemdativ added the enhancement New feature or request label Apr 10, 2020
calmandniceperson and others added 5 commits April 13, 2020 17:53
Create index for reportsig during initialization of DBConnection
8edf8a5
Return an API error if reportsig already exists
818050c
Add test for duplicates
ce1370c
Do nothing if the reportsig already exists
2b3e240 
After considering the consequences of returning an error (clients could check which IDs already exist), we decided to do nothing if a duplicate exists
Merge pull request #13 from ito-org/reportsig-duplicates
f4add66 
Handle duplicate reportsigs
@rettetdemdativ rettetdemdativ changed the title [WIP] Refactor backend for TCN compliance Refactor backend for TCN compliance Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@assert-not-singularity assert-not-singularity assert-not-singularity left review comments

@haveyaseen haveyaseen Awaiting requested review from haveyaseen

Assignees

@rettetdemdativ rettetdemdativ

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rettetdemdativ @assert-not-singularity