Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Two small improvements to the updated SPV mining alert #947

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

saivann commented Jul 6, 2015

No description provided.

saivann added some commits Jul 6, 2015

@harding harding commented on the diff Jul 6, 2015

_alerts/2015-07-04-spv-mining.html
@@ -9,7 +9,7 @@
update: 2015-07-06 02:00. All times are UTC.</em></p>
**Note: this alert is on-going: the situation has not yet been
-resolved. ([Update #1](#update-1))**
+resolved. ([Read more](#cause))**
@harding

harding Jul 6, 2015

Contributor

Here I kind of wanted to provide as easy way for people who had read previous versions of the alert to get straight to the new information.

@saivann

saivann Jul 6, 2015

Contributor

Please feel free to ignore that commit, but FWIW, I didn't notice what text it was linking to at a first glance due to (1) it being at the bottom of the page and (2) it not using any title, nothing very obvious.

@saivann

saivann Jul 6, 2015

Contributor

@harding Ah yes, probably!

@harding

harding Jul 6, 2015

Contributor

Cool. I cherry picked your other commit, so closing this PR. Thanks!

@harding harding closed this Jul 6, 2015

@saivann saivann deleted the saivann:spv-mining-update2 branch Jul 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment