Support Coinbase staking_reward tx type #7619
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #7619 +/- ##
===========================================
- Coverage 80.04% 71.43% -8.61%
===========================================
Files 1103 1103
Lines 106532 106532
Branches 13449 13449
===========================================
- Hits 85270 76101 -9169
- Misses 19120 28593 +9473
+ Partials 2142 1838 -304
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Left a comment about the change.
For the subtype, would need to check.
I think the interest/inflation_reward/staking_rewards can be like this.
But if you see the place you edited has another elif which is afaics (but I am not sure since I don't use coinbase and did not write the code) concerns internal movements/receivals. So that part should stay as is.
| @@ -538,7 +538,7 @@ def _query_single_account_transactions( | |||
| elif tx_type in ('buy', 'sell'): | |||
| self._process_normal_trade(event=transaction, trades=trades) | |||
| elif ( | |||
| tx_type in ('interest', 'inflation_reward') or | |||
There was a problem hiding this comment.
If this exists as a transaction type then it's a good addition.
Can you also edit the test at test_coinbase_query_income_loss_expense and add a mock event with this type and test it in the test?
When digging into coinbase API responses, I found that rotki ignored the
staking_rewardtx type. This PR adds support in the same way asinflation_rewardandinterest.As a related note, I don't know why all three (
interest,inflation_rewardandstaking_reward) are reported with Event Subtype NONE where they could be reported as REWARD. If you want me to change this as well with this PR, let me know.Checklist