-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Stablecoins page [WIP #1934] #1938
Conversation
FYI converted this PR to draft. |
Can we not use that feature as it prevents being able to check deploy previews on your phone 😕 |
Co-authored-by: Sam Richards <sbrichards@gmail.com>
Sure!
…Sent from my iPhone
On 7 Dec 2020, at 21:18, Sam Richards ***@***.***> wrote:
@ryancreatescopy this looks a bit awkward IMO. Would you be game to just change this to a text description, e.g. "Market capitalization is the total number of tokens that exist multiplied by the value per token"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall this looks terrific! See comment re: string translations. Please check out my market cap table additions. If all looks good we can deploy first thing tomorrow.
) | ||
const ALGORITHMIC = translateMessageId("page-stablecoins-algorithmic", intl) | ||
|
||
// TODO confirm type & url |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ryancreatescopy mind spot-checking me here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, but what am I spot-checking exactly 😬 😄
const ALGORITHMIC = translateMessageId("page-stablecoins-algorithmic", intl) | ||
|
||
// TODO confirm type & url | ||
const stablecoins = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samajammin where is this metadata being used?
Everything now looks good. Just one todo @samajammin
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ryancreatescopy I believe this is ready to go! Let's confirm the deploy preview looks good before merging.
Description
New stablecoins page – feedback welcome.
Related Issue
#1934