Hide signup when registrations are closed or the instance is private

src/shared/components/app/app.tsx

@@ -16,6 +16,7 @@ import "./styles.scss";
 import { Theme } from "./theme";
 import AnonymousGuard from "../common/anonymous-guard";
 import { CodeTheme } from "./code-theme";
+import { shouldHideSignup } from "@utils/helpers";
 
 interface AppProps {
   user?: MyUserInfo;
@@ -64,7 +65,12 @@ export class App extends Component<AppProps, any> {
             <Navbar siteRes={siteRes} />
             <div className="mt-4 p-0 fl-1">
               <Switch>
-                {routes.map(
+                {routes
+                  .filter(
+                    ({ path }) =>
+                      !(path === "/signup" && shouldHideSignup(siteRes)),
+                  )
+                  .map(
                     ({ path, component: RouteComponent, fetchInitialData }) => (
                       <Route
                         key={path}

src/shared/components/app/navbar.tsx

@@ -1,6 +1,6 @@
 import { showAvatars } from "@utils/app";
 import { isBrowser } from "@utils/browser";
-import { numToSI } from "@utils/helpers";
+import { numToSI, shouldHideSignup } from "@utils/helpers";
 import { amAdmin, canCreateCommunity } from "@utils/roles";
 import { Component, createRef, linkEvent } from "inferno";
 import { NavLink } from "inferno-router";
@@ -437,6 +437,7 @@ export class Navbar extends Component<NavbarProps, NavbarState> {
                       {I18NextService.i18n.t("login")}
                     </NavLink>
                   </li>
+                  {!shouldHideSignup(this.props.siteRes) && (
                     <li className="nav-item">
                       <NavLink
                         to="/signup"
@@ -447,6 +448,7 @@ export class Navbar extends Component<NavbarProps, NavbarState> {
                         {I18NextService.i18n.t("sign_up")}
                       </NavLink>
                     </li>
+                  )}
                 </>
               )}
             </ul>

src/shared/utils/helpers/index.ts

@@ -23,6 +23,7 @@ import validInstanceTLD from "./valid-instance-tld";
 import validTitle from "./valid-title";
 import validURL from "./valid-url";
 import dedupByProperty from "./dedup-by-property";
+import shouldHideSignup from "./should-hide-signup";
 
 export {
   capitalizeFirstLetter,
@@ -50,4 +51,5 @@ export {
   validTitle,
   validURL,
   dedupByProperty,
+  shouldHideSignup,
 };

src/shared/utils/helpers/should-hide-signup.ts

@@ -0,0 +1,8 @@
+import { GetSiteResponse } from "lemmy-js-client";
+
+export default function shouldHideSignup(siteRes?: GetSiteResponse) {
+  return (
+    siteRes?.site_view.local_site.registration_mode === "Closed" ||
+    siteRes?.site_view.local_site.private_instance
+  );
+}