Fix: rename Collectibles to NFTs (#3768)

5afe · Apr 7, 2022 · c8bf105 · c8bf105
1 parent 8e4d2a0
commit c8bf105
Show file tree

Hide file tree

Showing 8 changed files with 19 additions and 15 deletions.
diff --git a/src/components/AppLayout/Sidebar/useSidebarItems.tsx b/src/components/AppLayout/Sidebar/useSidebarItems.tsx
@@ -61,7 +61,7 @@ const useSidebarItems = (): ListItemType[] => {
       }),
       makeEntryItem({
         disabled: !isCollectiblesEnabled,
-        label: 'Collectibles',
+        label: 'NFTs',
         iconType: 'collectibles',
         href: currentSafeRoutes.ASSETS_BALANCES_COLLECTIBLES,
       }),

diff --git a/src/routes/routes.ts b/src/routes/routes.ts
@@ -47,8 +47,9 @@ export const LOAD_SAFE_ROUTE = generatePath(LOAD_SPECIFIC_SAFE_ROUTE) // By prov
 
 // [SAFE_SECTION_SLUG], [SAFE_SUBSECTION_SLUG] populated safe routes
 export const SAFE_ROUTES = {
-  ASSETS_BALANCES: `${ADDRESSED_ROUTE}/balances`, // [SAFE_SECTION_SLUG] === 'balances'
-  ASSETS_BALANCES_COLLECTIBLES: `${ADDRESSED_ROUTE}/balances/collectibles`, // [SAFE_SUBSECTION_SLUG] === 'collectibles'
+  ASSETS_BALANCES: `${ADDRESSED_ROUTE}/balances`,
+  ASSETS_BALANCES_COLLECTIBLES: `${ADDRESSED_ROUTE}/balances/nfts`,
+  LEGACY_COLLECTIBLES: `${ADDRESSED_ROUTE}/balances/collectibles`,
   TRANSACTIONS: `${ADDRESSED_ROUTE}/transactions`,
   TRANSACTIONS_HISTORY: `${ADDRESSED_ROUTE}/transactions/history`,
   TRANSACTIONS_QUEUE: `${ADDRESSED_ROUTE}/transactions/queue`,

diff --git a/src/routes/safe/components/Balances/Collectibles/index.tsx b/src/routes/safe/components/Balances/Collectibles/index.tsx
@@ -106,9 +106,7 @@ const Collectibles = (): React.ReactElement => {
     return (
       <Card className={classes.cardOuter}>
         <div className={classes.cardInner}>
-          <Paragraph className={classes.noData}>
-            {nftLoaded ? 'No collectibles available' : 'Loading collectibles...'}
-          </Paragraph>
+          <Paragraph className={classes.noData}>{nftLoaded ? 'No NFTs available' : 'Loading NFTs...'}</Paragraph>
         </div>
       </Card>
     )

diff --git a/src/routes/safe/components/Balances/SendModal/screens/ChooseTxType/index.tsx b/src/routes/safe/components/Balances/SendModal/screens/ChooseTxType/index.tsx
@@ -115,12 +115,8 @@ const ChooseTxType = ({
                 variant="contained"
                 testId="modal-send-collectible-btn"
               >
-                <Img
-                  alt="Send collectible"
-                  className={classNames(classes.leftIcon, classes.iconSmall)}
-                  src={Collectible}
-                />
-                Send collectible
+                <Img alt="Send NFT" className={classNames(classes.leftIcon, classes.iconSmall)} src={Collectible} />
+                Send NFT
               </Button>
             </Track>
           )}

diff --git a/src/routes/safe/components/Balances/SendModal/screens/ReviewCollectible/index.tsx b/src/routes/safe/components/Balances/SendModal/screens/ReviewCollectible/index.tsx
@@ -102,7 +102,7 @@ const ReviewCollectible = ({ onClose, onPrev, tx }: Props): React.ReactElement =
 
   return (
     <TxModalWrapper txData={txData} txTo={tx.assetAddress} onSubmit={submitTx} onBack={onPrev}>
-      <ModalHeader onClose={onClose} subTitle={getStepTitle(2, 2)} title="Send collectible" />
+      <ModalHeader onClose={onClose} subTitle={getStepTitle(2, 2)} title="Send NFT" />
       <Hairline />
       <Block className={classes.container}>
         <SafeInfo text="Sending from" />

diff --git a/src/routes/safe/components/Balances/SendModal/screens/SendCollectible/index.tsx b/src/routes/safe/components/Balances/SendModal/screens/SendCollectible/index.tsx
@@ -116,7 +116,7 @@ const SendCollectible = ({
 
   return (
     <>
-      <ModalHeader onClose={onClose} subTitle={getStepTitle(1, 2)} title="Send collectible" />
+      <ModalHeader onClose={onClose} subTitle={getStepTitle(1, 2)} title="Send NFT" />
       <Hairline />
       <GnoForm formMutators={formMutators} initialValues={initialValues} onSubmit={handleSubmit}>
         {(...args) => {

diff --git a/src/routes/safe/components/Balances/index.tsx b/src/routes/safe/components/Balances/index.tsx
@@ -22,7 +22,7 @@ export const BALANCE_ROW_TEST_ID = 'balance-row'
 
 enum SECTION_NAME {
   coins = 'Coins',
-  collectibles = 'Collectibles',
+  collectibles = 'NFTs',
 }
 
 const Balances = (): ReactElement => {

diff --git a/src/routes/safe/container/index.tsx b/src/routes/safe/container/index.tsx
@@ -89,6 +89,15 @@ const Container = (): React.ReactElement => {
   return (
     <>
       <Switch>
+        {/* Legacy redirect */}
+        <Route
+          path={SAFE_ROUTES.LEGACY_COLLECTIBLES}
+          exact
+          render={() => (
+            <Redirect to={generateSafeRoute(SAFE_ROUTES.ASSETS_BALANCES_COLLECTIBLES, extractPrefixedSafeAddress())} />
+          )}
+        />
+
         <Route
           exact
           path={[SAFE_ROUTES.ASSETS_BALANCES, SAFE_ROUTES.ASSETS_BALANCES_COLLECTIBLES]}