show asset name instead of symbol for setApprovalForAll calls on NFT contracts where possible (#15296)

feature/default_network_editable
Alex Donesky 2 years ago committed by seaona
parent 1d342252c5
commit 11abdddaa2
  1. 10
      ui/pages/confirm-approve/confirm-approve-content/confirm-approve-content.component.js

@ -453,6 +453,7 @@ export default class ConfirmApproveContent extends Component {
chainId,
assetStandard,
tokenSymbol,
isSetApproveForAll,
} = this.props;
const { t } = this.context;
let titleTokenDescription = t('token');
@ -479,7 +480,10 @@ export default class ConfirmApproveContent extends Component {
titleTokenDescription = unknownTokenLink;
}
if (assetStandard === ERC20 || (tokenSymbol && !tokenId)) {
if (
assetStandard === ERC20 ||
(tokenSymbol && !tokenId && !isSetApproveForAll)
) {
titleTokenDescription = tokenSymbol;
} else if (
assetStandard === ERC721 ||
@ -488,9 +492,9 @@ export default class ConfirmApproveContent extends Component {
(assetName && tokenId) ||
(tokenSymbol && tokenId)
) {
const tokenIdWrapped = tokenId ? ` (#${tokenId})` : null;
const tokenIdWrapped = tokenId ? ` (#${tokenId})` : '';
if (assetName || tokenSymbol) {
titleTokenDescription = `${assetName ?? tokenSymbol} ${tokenIdWrapped}`;
titleTokenDescription = `${assetName ?? tokenSymbol}${tokenIdWrapped}`;
} else {
const unknownNFTBlockExplorerLink = getTokenTrackerLink(
tokenAddress,

Loading…
Cancel
Save