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.

Sign up for GitHub

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

Jump to bottom

Patch/fee processing other bugs various csv #520

Merged
merged 5 commits into from
Jan 21, 2024
Merged

Patch/fee processing other bugs various csv #520

merged 5 commits into from
Jan 21, 2024

Conversation

pharr117
Copy link
Collaborator

Fixes a number of issues in the CSV codebase:

  1. Fees not being attached to the actual TX in Cointracker output
  2. Denom processing for IBC Ack and Recv messages
  3. Bad parsing of start/end time params leading to various date bugs

@danbryan danbryan merged commit 9665106 into main Jan 21, 2024
4 checks passed
@danbryan danbryan deleted the patch/fee-processing-other-bugs-various-csv branch January 21, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danbryan danbryan Awaiting requested review from danbryan

Assignees
No one assigned
Labels
grant OGP2024
Projects
Defiant OSS
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pharr117 @danbryan