-
Notifications
You must be signed in to change notification settings - Fork 38
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
feat: legal page #824
feat: legal page #824
Conversation
✅ Deploy Preview for tradetrust-mainnet ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
<a | ||
download={TRADETRUST_LEGAL_ARTICLE_TITLE} | ||
href="/static/images/legality/tradetrust-legal-article.pdf" | ||
onClick={downloadArticle} |
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.
Pretty sure the click count for this will be very high!😄
<meta | ||
name="keywords" | ||
content="Blockchain, NFT, Ethereum, Electronic Trade Document, Verifiable Document, Digital Trade Document, Transferable Documents, Electronic Transactions Act, Bill of Lading" |
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.
Do we have a parent component to house these common meta tags for all pages?
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.
if so i think i can refactor the meta data to here, then have a common "head" component. Or do you any other idea in mind?
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.
Just thought in case we have a parent component and missed it in this PR only, don’t think it’s necessary to implement it into this PR if we don’t already have one.
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.
LGTM, the <head><meta /></head>
can do on a separate PR
Summary
Legal analysis page
Changes
Issues
https://www.pivotaltracker.com/story/show/185938637