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
Indicate encrypted transactions in TX lists #505
Conversation
Deployed to Cloudflare Pages
|
3aa9803
to
911931c
Compare
Visuals validated by @donouwens |
911931c
to
a12e292
Compare
a12e292
to
b032346
Compare
@@ -57,3 +57,7 @@ export function getEthAccountAddress(preimage: AddressPreimage | undefined): str | |||
// We need to convert from base64 to hex, add the prefix, and convert to checksum address | |||
return toChecksumAddress(`0x${Buffer.from(preimage.address_data, 'base64').toString('hex')}`) | |||
} | |||
|
|||
export function uniq<T>(input: T[] | undefined): T[] { | |||
return input === undefined ? [] : [...new Set(input)] |
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.
not sure we need this function, but Ok. [...new Set(undefined)] will return an empty array anyway
...(canHaveEncryption | ||
? [{ content: (<LockIcon htmlColor={COLORS.grayMedium} />) as unknown as string }] | ||
: []), // The table does support widgets in the column headers, but the TS definition is unaware of that. |
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.
Table does not support it too well
explorer/src/app/components/Table/index.tsx
Line 108 in a77aa3a
key={column.content} |
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.
Yes, that could be improved. (By generating the key using the index, or something similar.)
As per https://app.clickup.com/t/861mx1bew and https://app.clickup.com/t/861mx1kwu
For testing, look for block 411186 on Sapphire testnet, or the account 0x6fA03516728d4a483578D63ea124C4Ec4A00e2bB