Merge pull request #1743 from RafaelTaranto/backport/explicitly-show-errors-on-tx-page

LAM-539 backport: explicitly show errors on tx page
This commit is contained in:
Rafael Taranto 2024-11-29 08:24:43 +00:00 committed by GitHub
commit 711535c476
3 changed files with 17 additions and 9 deletions

View file

@ -17,6 +17,7 @@ const CopyToClipboard = ({
buttonClassname, buttonClassname,
children, children,
wrapperClassname, wrapperClassname,
removeSpace = true,
...props ...props
}) => { }) => {
const [anchorEl, setAnchorEl] = useState(null) const [anchorEl, setAnchorEl] = useState(null)
@ -46,7 +47,8 @@ const CopyToClipboard = ({
{children} {children}
</div> </div>
<div className={classnames(classes.buttonWrapper, buttonClassname)}> <div className={classnames(classes.buttonWrapper, buttonClassname)}>
<ReactCopyToClipboard text={R.replace(/\s/g, '')(children)}> <ReactCopyToClipboard
text={removeSpace ? R.replace(/\s/g, '')(children) : children}>
<button <button
aria-describedby={id} aria-describedby={id}
onClick={event => handleClick(event)}> onClick={event => handleClick(event)}>

View file

@ -191,6 +191,13 @@ const DetailsRow = ({ it: tx, timezone }) => {
<> <>
<Label>Transaction status</Label> <Label>Transaction status</Label>
<span className={classes.bold}>{getStatus(tx)}</span> <span className={classes.bold}>{getStatus(tx)}</span>
{getStatusDetails(tx) ? (
<CopyToClipboard removeSpace={false} className={classes.errorCopy}>
{getStatusDetails(tx)}
</CopyToClipboard>
) : (
<></>
)}
</> </>
) )
@ -392,13 +399,7 @@ const DetailsRow = ({ it: tx, timezone }) => {
</div> </div>
<div className={classes.lastRow}> <div className={classes.lastRow}>
<div className={classes.status}> <div className={classes.status}>
{getStatusDetails(tx) ? ( {errorElements}
<HoverableTooltip parentElements={errorElements} width={200}>
<P>{getStatusDetails(tx)}</P>
</HoverableTooltip>
) : (
errorElements
)}
{((tx.txClass === 'cashOut' && getStatus(tx) === 'Pending') || {((tx.txClass === 'cashOut' && getStatus(tx) === 'Pending') ||
(tx.txClass === 'cashIn' && getStatus(tx) === 'Batched')) && ( (tx.txClass === 'cashIn' && getStatus(tx) === 'Batched')) && (
<ActionButton <ActionButton

View file

@ -1,7 +1,7 @@
import typographyStyles from 'src/components/typography/styles' import typographyStyles from 'src/components/typography/styles'
import { offColor, comet, white, tomato } from 'src/styling/variables' import { offColor, comet, white, tomato } from 'src/styling/variables'
const { p } = typographyStyles const { p, label3 } = typographyStyles
export default { export default {
wrapper: { wrapper: {
@ -137,5 +137,10 @@ export default {
}, },
swept: { swept: {
width: 250 width: 250
},
errorCopy: {
extend: label3,
lineBreak: 'normal',
maxWidth: 180
} }
} }