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

Adding Brazilian exchange Walltime. #1815

Merged
merged 1 commit into from
Dec 27, 2017
Merged

Conversation

felipelalli
Copy link
Contributor

No description provided.

@wbnns wbnns self-assigned this Sep 21, 2017
@wbnns wbnns added On Hold and removed Under Review labels Oct 7, 2017
@Mirobit
Copy link
Contributor

Mirobit commented Oct 10, 2017

Is there any update on their statement that they run Bitcoin Core?
If they continue to do, they should be added.

@felipelalli
Copy link
Contributor Author

@Mirobit we always will run Bitcoin Core. Do you need additional confirmation? I'm the Walltime CEO and you can verify it by seeing the GPG signature in my commit. Thank you!

@chacham18
Copy link

I confirm, exchange exists and is operated by @felipelalli as stated above.

@augustresende
Copy link

Me too.

Copy link
Contributor

@wbnns wbnns left a comment

Choose a reason for hiding this comment

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

@felipelalli Hello, there's just a small issue with a missing line break, which I've noted. Once you fix it, we can go ahead and get you added.

@@ -65,6 +65,7 @@ <h3 id="brazil"><img src="/img/flags/BR.png" alt="Brazilian flag">Brazil</h3>
<p>
<a href="https://foxbit.com.br/">Foxbit</a><br>
<a href="https://www.mercadobitcoin.com.br/">Mercado Bitcoin</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

A line break (<br>) is missing after Mercado Bitcoin. Without it, the listings will break - could you please fix?

@felipelalli
Copy link
Contributor Author

@wbnns I'm sorry, I just added and pushed force to fix.

@wbnns wbnns merged commit e1e92d9 into bitcoin-dot-org:master Dec 27, 2017
@wbnns
Copy link
Contributor

wbnns commented Dec 27, 2017

@felipelalli No problem, thanks!

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.

5 participants