Decouple Tower logic from database layer - Githubissues

Clinical-Genomics / trailblazer

Keep track of and manage analyses

MIT License

5 stars 2 forks source link

Decouple Tower logic from database layer #436

Open seallard opened 2 months ago

seallard commented 2 months ago

update.py contains logic related to the Tower API. It should be extracted from there. The Tower API is also instantiated per analysis. It should be refactored to only be instantiated once, it should also be split into two:

A TowerAPIClient which only handles the interactions with the REST API
A TowerService which wraps around the client, doing any necessary remapping to dtos/domain models