-
Notifications
You must be signed in to change notification settings - Fork 81
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
add genericscheme callData raw data to the proposal details view #2052
Conversation
<div> | ||
<strong>NFT Contract:</strong> <a href={linkToEtherScan(decodedCallData.values[1])} target="_blank" rel="noopener noreferrer">{decodedCallData.values[1]}</a> | ||
</div> | ||
<div> | ||
<strong>TokenID:</strong> {decodedCallData.values[2]} | ||
</div> | ||
<div> | ||
<strong>Raw call data:</strong> |
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.
Don't we prefer to eliminate the use of <strong>
and define the layout using div and CSS?
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.
I just didn't feel like this time refactoring and testing the entire thing, so just copied the pattern that was there
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.
On the code like this that is coming in from outside contributors, we can hopefully get them to follow these standards before accepting their PRs.
@@ -16,6 +17,14 @@ interface IProps { | |||
|
|||
export default class ProposalSummaryCO2ken extends React.Component<IProps, null> { | |||
|
|||
private rawCallData(proposalState: IGenericPluginProposalState) { | |||
return <> | |||
<p>Raw call data:</p> |
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.
Please use i18next
here and in the rest of the places.
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.
Done
Fixes #2021 for 2.0.