Merge pull request #8145 from sagemathinc/onprem-hide-account-balance

frontend: disable the balance button in account settings
sagemathinc · Jan 24, 2025 · ea0cb66 · ea0cb66
2 parents 8e82ece + bb80474
commit ea0cb66
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 7 deletions.
diff --git a/src/packages/frontend/account/account-page.tsx b/src/packages/frontend/account/account-page.tsx
@@ -229,7 +229,7 @@ export const AccountPage: React.FC = () => {
   function renderExtraContent() {
     return (
       <Space>
-        <BalanceButton />
+        {is_commercial ? <BalanceButton /> : undefined}
         <I18NSelector isWide={isWide} />
         <SignOut everywhere={false} highlight={true} narrow={!isWide} />
       </Space>

diff --git a/src/packages/frontend/purchases/balance-button.tsx b/src/packages/frontend/purchases/balance-button.tsx
@@ -23,6 +23,7 @@ export default function BalanceButton({
   topBar?: boolean;
 }) {
   const intl = useIntl();
+  const is_commercial = useTypedRedux("customize", "is_commercial");
   const [open, setOpen] = useState<boolean>(false);
   const [loading, setLoading] = useState<boolean>(true);
   const dbBalance = useTypedRedux("account", "balance");
@@ -126,10 +127,16 @@ export default function BalanceButton({
     );
   }
 
-  return (
-    <>
-      {renderDisplay()}
-      {renderModal()}
-    </>
-  );
+  // This ensures it only shows up in commercial setups.
+  // Wherever it is used, the component shouldn't be instantiated in those cases, though.
+  if (!is_commercial) {
+    return;
+  } else {
+    return (
+      <>
+        {renderDisplay()}
+        {renderModal()}
+      </>
+    );
+  }
 }