Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add TX to fee. Parse fees that were not found through event parsing #518

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

pharr117
Copy link
Collaborator

@pharr117 pharr117 commented Jan 5, 2024

Gathers fees that were not included in a taxable tx in all CSV parsers. Fixes a bug where fees were missing if the address did not actually participate in the sends/receives of the messages (such as a relayer relaying IBC messages for addresses).

Closes #512

@pharr117 pharr117 requested a review from danbryan January 5, 2024 02:54
@danbryan danbryan merged commit 96faf11 into main Jan 5, 2024
4 checks passed
@danbryan danbryan deleted the patch/512-fees-missing-when-no-taxable-tx branch January 5, 2024 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fees are not being CSV processed when address was TX signer but did not send/receive in the TX
2 participants