opened
thomasleese
username_0
This refactors the core Asset Manger workers into services making it clearer what their responsibilities are. This should hopefully make the code easier to understand to new people.

The next part will be to change the attachment workers into services, which will make the code clearer but also reduce performance by not requiring a database look up for each subworker in the data update worker. I've decided to do that in a separate PR to make them easier to review.

[Trello Card](https://trello.com/c/1sd6Ve2a/292-investigate-and-reduce-daily-asset-job-spike)
Refactor core Asset Manager Workers
issue
created
thomasleese
username_0
@username_1 Tests pass on this now.
comment
created
barrucadu
username_1
I always feel a bit sad when I refactor something to be clearer, and the number of lines goes up.
comment
created
thomasleese
username_0
Haha yeah, same... maybe after my next refactor it'll go down...
comment