Skip to content

Commit

Permalink
Merge branch 'develop' into 17191/onboarding-unit-tests
Browse files Browse the repository at this point in the history
  • Loading branch information
tmashuang authored Jan 23, 2023
2 parents 375bcaa + cf10b2a commit e6d1868
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 5 deletions.
7 changes: 6 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Fixed
- Added displaying the primaryType during Typed data signing ([#17077](https://github.com/MetaMask/metamask-extension/pull/17077))

## [10.24.1]
### Added
- Ensure app name appears for Taiwanese language speakers in the extension stores ([#17304](https://github.com/MetaMask/metamask-extension/pull/17304))

## [10.24.0]
### Added
- Add NFT setApprovalForAll warning popover when approving the NFT Colleciton ([#16195](https://github.com/MetaMask/metamask-extension/pull/16195))
Expand Down Expand Up @@ -3425,7 +3429,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
### Uncategorized
- Added the ability to restore accounts from seed words.

[Unreleased]: https://github.com/MetaMask/metamask-extension/compare/v10.24.0...HEAD
[Unreleased]: https://github.com/MetaMask/metamask-extension/compare/v10.24.1...HEAD
[10.24.1]: https://github.com/MetaMask/metamask-extension/compare/v10.24.0...v10.24.1
[10.24.0]: https://github.com/MetaMask/metamask-extension/compare/v10.23.3...v10.24.0
[10.23.3]: https://github.com/MetaMask/metamask-extension/compare/v10.23.2...v10.23.3
[10.23.2]: https://github.com/MetaMask/metamask-extension/compare/v10.23.1...v10.23.2
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "metamask-crx",
"version": "10.24.0",
"version": "10.24.1",
"private": true,
"repository": {
"type": "git",
Expand Down
5 changes: 4 additions & 1 deletion ui/components/app/collectible-details/collectible-details.js
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,10 @@ export default function CollectibleDetails({ collectible }) {
className="collectible-details__card"
>
{image ? (
<img className="collectible-details__image" src={image} />
<img
className="collectible-details__image"
src={collectibleImageURL}
/>
) : (
<CollectibleDefaultImage name={name} tokenId={tokenId} />
)}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,15 @@
/* eslint-disable no-negated-condition */
import React, { useState } from 'react';
import { useSelector } from 'react-redux';

import PropTypes from 'prop-types';
import classnames from 'classnames';

import { TransactionType } from '../../../../../../shared/constants/transaction';
import { toChecksumHexAddress } from '../../../../../../shared/modules/hexstring-utils';
import { useI18nContext } from '../../../../../hooks/useI18nContext';
import useAddressDetails from '../../../../../hooks/useAddressDetails';
import { getIpfsGateway } from '../../../../../selectors';

import Identicon from '../../../../ui/identicon';
import InfoTooltip from '../../../../ui/info-tooltip';
Expand All @@ -15,6 +18,7 @@ import Typography from '../../../../ui/typography';
import { TYPOGRAPHY } from '../../../../../helpers/constants/design-system';
import { ORIGIN_METAMASK } from '../../../../../../shared/constants/app';
import SiteOrigin from '../../../../ui/site-origin';
import { getAssetImageURL } from '../../../../../helpers/utils/util';

const ConfirmPageContainerSummary = (props) => {
const {
Expand All @@ -35,6 +39,7 @@ const ConfirmPageContainerSummary = (props) => {

const [showNicknamePopovers, setShowNicknamePopovers] = useState(false);
const t = useI18nContext();
const ipfsGateway = useSelector(getIpfsGateway);

const contractInitiatedTransactionType = [
TransactionType.contractInteraction,
Expand Down Expand Up @@ -62,12 +67,14 @@ const ConfirmPageContainerSummary = (props) => {
const checksummedAddress = toChecksumHexAddress(contractAddress);

const renderImage = () => {
const imagePath = getAssetImageURL(image, ipfsGateway);

if (image) {
return (
<img
className="confirm-page-container-summary__icon"
width={36}
src={image}
src={imagePath}
/>
);
} else if (contractAddress) {
Expand All @@ -76,7 +83,6 @@ const ConfirmPageContainerSummary = (props) => {
className="confirm-page-container-summary__icon"
diameter={36}
address={contractAddress}
image={image}
/>
);
}
Expand Down

0 comments on commit e6d1868

Please sign in to comment.