Technophobes / zoo

0 stars 0 forks source link

Decouple api from database #10

Open scottzec opened 4 years ago

scottzec commented 4 years ago

https://github.com/Technophobes/zoo/blob/d9787aa7d4999a7bf92b39b622d5ed2fca8efb64/api.py#L41

L41 and L42 expect too much understanding of sqlalchemy

Discuss a way to make the api less integrated with the database Andrew doesn't like the use of dict here He suggests that there could be a way of doing this within the Class Perhaps a Class method way of doing this Is there a way that we can call a class method that does this much more nicely

An implementation that could be used: https://gist.github.com/mooperd/f0610ab1cafd6ddec0981a331bbe71bb

Will be necessary to make changes to all existing apis All GET requests will now be different It won't change POST Affects how we are extracting data, not how we are putting data in.