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

Address Aaron's comments in #4943 #4987

Closed
stringhandler opened this issue Dec 1, 2022 · 2 comments
Closed

Address Aaron's comments in #4943 #4987

stringhandler opened this issue Dec 1, 2022 · 2 comments

Comments

@stringhandler
Copy link
Collaborator

@AaronFeickert added some comments to #4943 after it was merged. Making a tracking issue so that they are not forgotten about
@SWvheerden SWvheerden mentioned this issue Dec 5, 2022
Merged
@SWvheerden
Copy link
Collaborator

Spelling address here: #4996

stringhandler pushed a commit that referenced this issue Dec 5, 2022
@SWvheerden
chore: fix spelling mistake in export of utxos to csv (#4996)
cc2c0d2 
Description
---
Fixes the spelling mistake in the export of utxo to csv function
Add missing fields of utxo

Motivation and Context
---
Addresses one of the issues here: #4987

How Has This Been Tested?
---
Manual
@AaronFeickert
Copy link
Collaborator

Given the discussion in the PR comments, I'm fine closing this issue if the reverse-domain suggestion is deemed unnecessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@SWvheerden @stringhandler @AaronFeickert