cowprotocol / token-imbalances

2 stars 0 forks source link

Refactoring #44

Closed shubhagarwal03 closed 2 months ago

shubhagarwal03 commented 2 months ago

This PR makes things more de-coupled. For instance, since gnosis chain prices are not available currently via price providers, we can now set a configuration in daemon.py by setting process_prices to False. Similarly for arbitrum_one.

Overall, the process_single_transaction function is more clear, and entries to the table are done at the end of all computations to ensure that if any issues in either computations arise, then it is added as an unprocessed transaction.