<p>Thanks for all that feedback, <a class="user-mention" data-hovercard-user-id="301810" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/cbeams">@cbeams</a>. Last 2 commits integrate all of your suggestions.</p>
<p>As mentioned in the last commit, all screenshots are placeholders (I need to do another transaction to get some screens). But the body text should be done for the most part.</p>
<p>Some questions:</p>
<ul>
<li>
<p>When discussing the fund-the-trade-wallet step, should we specify a minimum miner fee? I thought I remember seeing 20 sat/b recommended somewhere in Bisq, but I can't find it anymore.</p>
</li>
<li>
<p>I added a warning about centralized exchanges in a warning admonition. Ragged pretty hard on Coinbase. Many people seem to assume there's some inextricable link between Coinbase and bitcoin, so I figured it would help to call them out here. But maybe mentioning others would help? Or maybe it's throwing too much at the user and we should just remove the whole warning block.</p>
</li>
<li>
<p>Is there a preference/guideline on linking to outside companies/organizations? I think it would be nice to link to some places to help people find ATMs, provide more details on vouchers, find meetups, etc. If you have go-to sources you trust for any of those, let me know.</p>
</li>
</ul>
<p>I'm going to test this with my mom this weekend. Getting her some bitcoin for Mother's Day...only catch is she has to use these instructions to actually carry out the transaction :)</p>
<p>Will get updated screenshots in the process, so this thing should be pretty close to complete by the end of the weekend.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq-docs/pull/45#issuecomment-388458085">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZthH6I_Czlr5JHjuHQubOqV6Paxvqks5txeGdgaJpZM4TxXG2">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtiqUY6Fm0SBi30ri-SRyaDu04leYks5txeGdgaJpZM4TxXG2.gif" height="1" width="1" alt="" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/bisq-network/bisq-docs/pull/45#issuecomment-388458085"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bisq-docs","title":"bisq-network/bisq-docs","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bisq-docs"}},"updates":{"snippets":[{"icon":"PERSON","message":"@m52go in #45: Thanks for all that feedback, @cbeams. Last 2 commits integrate all of your suggestions.\r\n\r\nAs mentioned in the last commit, all screenshots are placeholders (I need to do another transaction to get some screens). But the body text should be done for the most part.\r\n\r\nSome questions:\r\n\r\n- When discussing the fund-the-trade-wallet step, should we specify a minimum miner fee? I thought I remember seeing 20 sat/b recommended somewhere in Bisq, but I can't find it anymore.\r\n\r\n- I added a warning about centralized exchanges in a warning admonition. Ragged pretty hard on Coinbase. Many people seem to assume there's some inextricable link between Coinbase and bitcoin, so I figured it would help to call them out here. But maybe mentioning others would help? Or maybe it's throwing too much at the user and we should just remove the whole warning block.\r\n\r\n- Is there a preference/guideline on linking to outside companies/organizations? I think it would be nice to link to some places to help people find ATMs, provide more details on vouchers, find meetups, etc. If you have go-to sources you trust for any of those, let me know.\r\n\r\nI'm going to test this with my mom this weekend. Getting her some bitcoin for Mother's Day...only catch is she has to use these instructions to actually carry out the transaction :)\r\n\r\nWill get updated screenshots in the process, so this thing should be pretty close to complete by the end of the weekend."}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq-docs/pull/45#issuecomment-388458085"}}}</script>
<script type="application/ld+json">{"@type":"MessageCard","@context":"http://schema.org/extensions","hideOriginalBody":"false","originator":"37567f93-e2a7-4e2a-ad37-a9160fc62647","title":"Re: [bisq-network/bisq-docs] [WIP] Produce a 'getting started' guide (#45)","sections":[{"text":"","activityTitle":"**Steve Jain**","activityImage":"https://avatars0.githubusercontent.com/u/735155?s=160\u0026v=4","activitySubtitle":"@m52go","facts":[]}],"potentialAction":[{"name":"Add a comment","@type":"ActionCard","inputs":[{"isMultiLine":true,"@type":"TextInput","id":"IssueComment","isRequired":false}],"actions":[{"name":"Comment","@type":"HttpPOST","target":"https://api.github.com","body":"{\"commandName\":\"IssueComment\",\"repositoryFullName\":\"bisq-network/bisq-docs\",\"issueId\":45,\"IssueComment\":\"{{IssueComment.value}}\"}"}]},{"name":"Close pull request","@type":"HttpPOST","target":"https://api.github.com","body":"{\"commandName\":\"PullRequestClose\",\"repositoryFullName\":\"bisq-network/bisq-docs\",\"pullRequestId\":45}"},{"targets":[{"os":"default","uri":"https://github.com/bisq-network/bisq-docs/pull/45#issuecomment-388458085"}],"@type":"OpenUri","name":"View on GitHub"},{"name":"Unsubscribe","@type":"HttpPOST","target":"https://api.github.com","body":"{\"commandName\":\"MuteNotification\",\"threadId\":331706806}"}],"themeColor":"26292E"}</script>