Skip to content

Conversation

@harding
Copy link
Collaborator

@harding harding commented Sep 17, 2018

Todo:

  • Optional: add Dashboard Items section (@moneyball)
  • Maybe describe Bitcoin Core 7965 given this week's merge of 14168? See note in the Notable Merges section tagged TODO. In any case, remove the TODO note. (@jnewbery)

@moneyball
Copy link
Contributor

moneyball commented Sep 17, 2018 via email

@jnewbery
Copy link
Contributor

ACK e89c9fb. I've added a new commit that:

  • describes bitcoin core issue 7965
  • softens the language around removing P2P reject messages.

I plan to send the newsletter out at ~9 eastern tomorrow. If anyone has the chance to review before I send it, it would be appreciated!

@harding
Copy link
Collaborator Author

harding commented Sep 18, 2018

8d52f86 text LGTM.

I've added information to the newsletter about the security release of Bitcoin Core 0.16.3.

@jnewbery
Copy link
Contributor

ACK 9dc561d

@jnewbery jnewbery merged commit d975b5d into bitcoinops:master Sep 18, 2018
long-term effort to disentangle the wallet code from the server code. Doing so
provides a number of benefits including easier code maintenance, better
opportunities for testing individual components, and potentially more secure
software if the wallet component is moved to its own process.

Choose a reason for hiding this comment

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

In the future could link to bitcoin/bitcoin#10102 when referring to multiprocess stuff, since that's where the implementation is. TBH though bitcoin/bitcoin#7965 doesn't actually have much (or anything?) to do with testing or process separation.

bitschmidty pushed a commit that referenced this pull request Aug 9, 2023
* newsletter-260: translate into Chinese

* Update 2023-07-19-newsletter.md

- Improved impression
- Accuracy improvement of paragraphs related to code running time

* Update 10-get-involved.md

- Expression improved

* Apply suggestions from code review

Co-authored-by: editor-Ajian <34365188+editor-Ajian@users.noreply.github.com>

---------

Co-authored-by: freeyao <johnyao1526@gmail.com>
Co-authored-by: editor-Ajian <34365188+editor-Ajian@users.noreply.github.com>
harding pushed a commit to harding/bitcoinops.github.io that referenced this pull request Sep 27, 2024
* newsletter-260: translate into Chinese

* Update 2023-07-19-newsletter.md

- Improved impression
- Accuracy improvement of paragraphs related to code running time

* Update 10-get-involved.md

- Expression improved

* Apply suggestions from code review

Co-authored-by: editor-Ajian <34365188+editor-Ajian@users.noreply.github.com>

---------

Co-authored-by: freeyao <johnyao1526@gmail.com>
Co-authored-by: editor-Ajian <34365188+editor-Ajian@users.noreply.github.com>
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.

4 participants