opentimestamps / opentimestamps.org

OpenTimestamps.org website
https://opentimestamps.org
24 stars 22 forks source link

Make the warning message for pending attestation a little bit more educational and informative #55

Closed FrancisPouliot closed 4 years ago

FrancisPouliot commented 4 years ago

To make it easier to understand what is going on for people who don't really know that much about OTS.

The text is

The Bitcoin transaction is unconfirmed. The attestation is still pending. Once the transaction confirms, this file will be updated.

petertodd commented 4 years ago

Concept ACK

Looks like an improvement to me, so long as the new text fits in the UI.

On June 29, 2020 10:59:56 PM EDT, Francis Pouliot notifications@github.com wrote:

To make it easier to understand what is going on for people who don't really know that much about OTS.

The text is

The Bitcoin transacation is unconfirmed. The attestation is still pending. Once the transaction confirms, this file will be updated. You can view, comment on, or merge this pull request online at:

https://github.com/opentimestamps/opentimestamps.org/pull/55

-- Commit Summary --

  • Update index.js

-- File Changes --

M assets/javascripts/index.js (2)

-- Patch Links --

https://github.com/opentimestamps/opentimestamps.org/pull/55.patch https://github.com/opentimestamps/opentimestamps.org/pull/55.diff

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/opentimestamps/opentimestamps.org/pull/55

RCasatta commented 4 years ago

LGTM, merging, will be updated in next site deploy