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

Newsletters: add 132 (2021-01-20) #506

Merged
merged 4 commits into from Jan 20, 2021

Conversation

harding
Copy link
Contributor

@harding harding commented Jan 18, 2021

Copy link
Collaborator

@jonatack jonatack left a comment

Choose a reason for hiding this comment

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

Seems there was an update while I was reviewing -- first read-through!

_posts/en/newsletters/2021-01-20-newsletter.md Outdated Show resolved Hide resolved
_posts/en/newsletters/2021-01-20-newsletter.md Outdated Show resolved Hide resolved
_posts/en/newsletters/2021-01-20-newsletter.md Outdated Show resolved Hide resolved
_posts/en/newsletters/2021-01-20-newsletter.md Outdated Show resolved Hide resolved
_posts/en/newsletters/2021-01-20-newsletter.md Outdated Show resolved Hide resolved
Copy link
Contributor

@jnewbery jnewbery left a comment

Choose a reason for hiding this comment

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

Looks good to me. Just one bad link.

_posts/en/newsletters/2021-01-20-newsletter.md Outdated Show resolved Hide resolved
@jnewbery
Copy link
Contributor

Pushed my write up of Bitcoin Core 19937.

Copy link
Contributor

@bitschmidty bitschmidty left a comment

Choose a reason for hiding this comment

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

Looking good so far, team

Copy link
Collaborator

@murchandamus murchandamus left a comment

Choose a reason for hiding this comment

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

LGTM

@harding
Copy link
Contributor Author

harding commented Jan 20, 2021

Made edits and added topic entries. Also reviewed @dongcarl, @bitschmidty, and @jnewbery's contributions---they look great. Thanks all!

@jnewbery
Copy link
Contributor

I've done a final review and all looks good to me. I've also pushed a small commit that adds a couple more links to the topics pages.

@harding
Copy link
Contributor Author

harding commented Jan 20, 2021

ACK 9843db2

@bitschmidty bitschmidty merged commit 1e72cac into bitcoinops:master Jan 20, 2021
@bitschmidty
Copy link
Contributor

Merged! Thanks @harding @dongcarl @jnewbery @xekyo @jonatack for your contributions and reviews this week

@jnewbery
Copy link
Contributor

Another great newsletter. Thanks team!

@jnewbery jnewbery mentioned this pull request Mar 31, 2021
7 tasks
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.

None yet

6 participants