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

Attempt to document the process of editing a batch transaction csv file. #146

Merged
merged 6 commits into from
Oct 22, 2018

Conversation

KanoczTomas
Copy link
Contributor

It can be used as a reference for new people who would work on the issues or for future self :)

@KanoczTomas KanoczTomas requested a review from cbeams October 20, 2018 19:30
@KanoczTomas KanoczTomas changed the title My attempt to document the process of editing a batch transaction csv file. Attempt to document the process of editing a batch transaction csv file. Oct 20, 2018
Copy link
Contributor

@cbeams cbeams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, @KanoczTomas. Could you also add a README to the root dir and link to this doc from there? Will make it more discoverable. We might want to move this to docs.bisq.network at some point as well, but it's quite fine to leave it here for now.

@cbeams
Copy link
Contributor

cbeams commented Oct 21, 2018

Also, I'd recommend qualifying the name of the doc as reimbursement-process.md, as the support repository is not necessarily only about reimbursements (although in practice that is pretty much the only thing that happens here).

Copy link
Contributor

@cbeams cbeams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@cbeams cbeams merged commit 87c9b99 into bisq-network:master Oct 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants