fix: _consume_uphold_transaction

[gahrb]

Closes #(issue_number)

Checklist

• [ ] The PR modified the frontend, and updated the user guide to reflect the changes.

[CLAassistant]

All committers have signed the CLA.

[yabirgb]

Hello @gahrb ! We have a CLA that you would need to sign before contributing to the repo. Can you provide an explanation to the PR to understand the issue?

[gahrb]

Thanks @LefterisJP and @yabirgb for getting back this quickly. I'm without my laptop until Tuesday and will get back to you afterwards. Marking it as draft for now.

[gahrb]

Hi @yabirgb & @LefterisJP,

I changed the commit to match your style guides and signed the CLA.

About the pull request:

• I found the data_import modules to be underrepresented in the tests
• At the same time they all seem to be doing the same: Take some exchange's export, bring it into rotki's data format and import the data

Since I haven't contributed yet to rotki, I found this a good and easy point of entrance. I picked uphold as an example and added the following changes:

• Create helper functions that can be used by each exchange's import module
• Call the corresponding functions (as a first step only) by uphold.

Let me know if this is of help and if the changes lgty. I would then mark the PR as ready.

[LefterisJP]

This PR has stayed open for too long without any response from the author. Will close it soon if no progress is made.

[LefterisJP]

Closing due to inactivity.