Skip to content

Commit

Permalink
fix: do not break page when mediaUrl is undefined
Browse files Browse the repository at this point in the history
  • Loading branch information
judith-Near committed Jun 20, 2022
1 parent 7f92c71 commit 192eacc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 5 additions & 5 deletions packages/frontend/src/components/nft/NFTMedia.js
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
import React, { useMemo } from "react";
import React, { useMemo } from 'react';

import FailedToLoad from "../../images/failed_to_load.svg";
import FailedToLoad from '../../images/failed_to_load.svg';

export function NFTMedia({ mediaUrl, autoPlay = false }) {
const [isVideo, mimeType] = useMemo(() => {
let mimeType;
// check mediaUrl string for .webm or .mp4 endings (case-insensitive)
if (mediaUrl && mediaUrl.match(/\.webm$/i)) mimeType = "webm";
else if (mediaUrl && mediaUrl.match(/\.mp4$/i)) mimeType = "mp4";
if (mediaUrl && mediaUrl.match(/\.webm$/i)) mimeType = 'webm';
else if (mediaUrl && mediaUrl.match(/\.mp4$/i)) mimeType = 'mp4';
// if there is a mediaUrl and a truthy mimeType (webm or mp4), we have a video
const isVideo = !!mediaUrl && mimeType;
return [isVideo, mimeType];
Expand All @@ -23,7 +23,7 @@ export function NFTMedia({ mediaUrl, autoPlay = false }) {
onError={(e) => {
e.target.onerror = null;
e.target.parentElement.setAttribute(
"poster",
'poster',
FailedToLoad
);
}}
Expand Down
2 changes: 1 addition & 1 deletion packages/frontend/src/hooks/fungibleTokensIncludingNEAR.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import compare from '../utils/compare';


export const useFungibleTokensIncludingNEAR = function ({ showTokensWithZeroBalance = false, includeNearContractName = false } = {}) {
const NEARAsTokenWithMetadata = useSelector(state => selectNEARAsTokenWithMetadata(state, {includeNearContractName}));
const NEARAsTokenWithMetadata = useSelector((state) => selectNEARAsTokenWithMetadata(state, {includeNearContractName}));
const accountId = useSelector(selectActiveAccountId);
const fungibleTokens = useSelector((state) =>
selectTokensWithMetadataForAccountId(state, { accountId, showTokensWithZeroBalance })
Expand Down

0 comments on commit 192eacc

Please sign in to comment.