csuttner
commented
3 years ago Closing as this is too many different issues tackled with one PR, it should be broken up and organized. Plus there were merge conflicts
Closed csuttner closed 3 years ago
csuttner
commented
3 years ago Closing as this is too many different issues tackled with one PR, it should be broken up and organized. Plus there were merge conflicts
Incorporates changes from ef-docker_repair branch to build and run the api container without errors (thank you).
Modifies twitter.py to return GET output data (in dictionary form) in the newly updated structure (see wiki). Apparently, dictionaries returned by flask will be converted to json automatically. We were getting extra backslashes due to this implicit conversion being compounded with our own json.dumps().
Modifies main.py, importantly but somewhat inconveniently, setting debug=False. If debug=True, will return json "pretty printed" with newlines and whitespace. Note - the app works fine with json in a pretty printed format.
Modifying main.py required rebuilding the container as it was not mounted as a volume. We might want to look into adjusting this.