Skip to content

Commit

Permalink
update
Browse files Browse the repository at this point in the history
  • Loading branch information
jpuri committed Oct 4, 2024
2 parents 632cbb0 + 4093fdf commit 8b643f6
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 11 deletions.
29 changes: 22 additions & 7 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -94,10 +94,25 @@ app/components/Views/NftDetails @MetaMask/metamask-assets
app/reducers/collectibles @MetaMask/metamask-assets

# UX Team
app/components @MetaMask/wallet-ux
app/reducers/experimentalSettings @MetaMask/wallet-ux
app/reducers/modals @MetaMask/wallet-ux
app/reducers/navigation @MetaMask/wallet-ux
app/reducers/onboarding @MetaMask/wallet-ux
app/reducers/privacy @MetaMask/wallet-ux
app/reducers/settings @MetaMask/wallet-ux
app/components/Views/AccountActions @MetaMask/wallet-ux
app/components/Views/AccountSelector @MetaMask/wallet-ux
app/components/Views/AddressQRCode @MetaMask/wallet-ux
app/components/Views/EditAccountName @MetaMask/wallet-ux
app/components/Views/LockScreen @MetaMask/wallet-ux
app/components/Views/Login @MetaMask/wallet-ux
app/components/Views/NetworkConnect @MetaMask/wallet-ux
app/components/Views/NetworkSelector @MetaMask/wallet-ux
app/components/Views/Onboarding @MetaMask/wallet-ux
app/components/Views/OnboardingCarousel @MetaMask/wallet-ux
app/components/Views/OnboardingSuccess @MetaMask/wallet-ux
app/components/Views/QRAccountDisplay @MetaMask/wallet-ux
app/components/Views/QRScanner @MetaMask/wallet-ux
app/components/Views/Settings @MetaMask/wallet-ux
app/components/Views/TermsAndConditions @MetaMask/wallet-ux

app/reducers/experimentalSettings @MetaMask/wallet-ux
app/reducers/modals @MetaMask/wallet-ux
app/reducers/navigation @MetaMask/wallet-ux
app/reducers/onboarding @MetaMask/wallet-ux
app/reducers/privacy @MetaMask/wallet-ux
app/reducers/settings @MetaMask/wallet-ux
5 changes: 2 additions & 3 deletions app/components/UI/SimulationDetails/useBalanceChanges.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,13 +98,12 @@ const fetchTokenExchangeRates = async (
chainId: Hex,
) => {
try {
const x = await fetchTokenContractExchangeRates({
return await fetchTokenContractExchangeRates({
tokenPricesService: new CodefiTokenPricesServiceV2(),
nativeCurrency,
tokenAddresses,
chainId,
});
return x;
} catch (err) {
return {};
}
Expand Down Expand Up @@ -201,7 +200,7 @@ export default function useBalanceChanges(
[JSON.stringify(erc20TokenAddresses), chainId, fiatCurrency],
);

if (erc20Decimals.pending || erc20FiatRates.pending || !simulationData) {
if (erc20Decimals.pending || erc20FiatRates.pending || !simulationData ) {
return { pending: true, value: [] };
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -593,7 +593,7 @@ class TransactionReview extends PureComponent {
primaryCurrency={primaryCurrency}
chainId={chainId}
/>
{useTransactionSimulations && (
{useTransactionSimulations && transactionSimulationData && (
<View style={styles.transactionSimulations}>
<SimulationDetails
simulationData={transactionSimulationData}
Expand Down

0 comments on commit 8b643f6

Please sign in to comment.