Switch to growthbook feature flags

Author: nabramow

app/backend/feature-flags.dev.json

@@ -1,12 +1,13 @@
 {
-  "test_growthbook_integration": true,
-  "sms_enabled": false,
-  "strong_verification_enabled": false,
-  "log_native_ota_requests": true,
-  "donations_enabled": false,
   "antibot_enabled": false,
-  "postal_verification_enabled": false,
+  "donations_enabled": false,
+  "email_ics_attachments_enabled": true,
   "listmonk_enabled": false,
+  "log_native_ota_requests": true,
   "notification_translations_enabled": true,
-  "email_ics_attachments_enabled": true
+  "postal_verification_enabled": false,
+  "public_trips_enabled": true,
+  "sms_enabled": false,
+  "strong_verification_enabled": false,
+  "test_growthbook_integration": true
 }

app/backend/src/couchers/servicers/public_trips.py

@@ -89,6 +89,9 @@ class PublicTrips(public_trips_pb2_grpc.PublicTripsServicer):
     def CreatePublicTrip(
         self, request: public_trips_pb2.CreatePublicTripReq, context: CouchersContext, session: Session
     ) -> public_trips_pb2.PublicTrip:
+        if not context.get_boolean_value("public_trips_enabled", False):
+            context.abort_with_error_code(grpc.StatusCode.UNAVAILABLE, "public_trips_disabled")
+
         user = session.execute(select(User).where(User.id == context.user_id)).scalar_one()
         if not has_completed_profile(session, user):
             context.abort_with_error_code(grpc.StatusCode.FAILED_PRECONDITION, "incomplete_profile_create_public_trip")
@@ -174,6 +177,9 @@ def CreatePublicTrip(
     def GetPublicTrip(
         self, request: public_trips_pb2.GetPublicTripReq, context: CouchersContext, session: Session
     ) -> public_trips_pb2.PublicTrip:
+        if not context.get_boolean_value("public_trips_enabled", False):
+            context.abort_with_error_code(grpc.StatusCode.UNAVAILABLE, "public_trips_disabled")
+
         trip_node_id = session.execute(
             select(PublicTrip.node_id).where(PublicTrip.id == request.trip_id)
         ).scalar_one_or_none()
@@ -196,6 +202,9 @@ def GetPublicTrip(
     def ListPublicTrips(
         self, request: public_trips_pb2.ListPublicTripsReq, context: CouchersContext, session: Session
     ) -> public_trips_pb2.ListPublicTripsRes:
+        if not context.get_boolean_value("public_trips_enabled", False):
+            context.abort_with_error_code(grpc.StatusCode.UNAVAILABLE, "public_trips_disabled")
+
         page_size = min(MAX_PAGINATION_LENGTH, request.page_size or MAX_PAGINATION_LENGTH)
         next_page_id = int(request.page_token) if request.page_token else 0
 
@@ -227,6 +236,9 @@ def ListPublicTrips(
     def ListPublicTripsByUser(
         self, request: public_trips_pb2.ListPublicTripsByUserReq, context: CouchersContext, session: Session
     ) -> public_trips_pb2.ListPublicTripsByUserRes:
+        if not context.get_boolean_value("public_trips_enabled", False):
+            context.abort_with_error_code(grpc.StatusCode.UNAVAILABLE, "public_trips_disabled")
+
         page_size = min(MAX_PAGINATION_LENGTH, request.page_size or MAX_PAGINATION_LENGTH)
         cursor_date, cursor_id = _parse_page_token(request.page_token)
         ascending = request.ascending
@@ -299,6 +311,9 @@ def ListPublicTripsByUser(
     def UpdatePublicTrip(
         self, request: public_trips_pb2.UpdatePublicTripReq, context: CouchersContext, session: Session
     ) -> public_trips_pb2.PublicTrip:
+        if not context.get_boolean_value("public_trips_enabled", False):
+            context.abort_with_error_code(grpc.StatusCode.UNAVAILABLE, "public_trips_disabled")
+
         public_trip = session.execute(select(PublicTrip).where(PublicTrip.id == request.trip_id)).scalar_one_or_none()
 
         if not public_trip or public_trip.user_id != context.user_id:

app/web/features/communities/CommunityPage/CommunityPage.tsx

@@ -1,3 +1,4 @@
+import { useFeatureValue } from "@growthbook/growthbook-react";
 import { styled, Typography } from "@mui/material";
 import HtmlMeta from "components/HtmlMeta";
 import EditCommunityPage from "features/communities/EditCommunityInfoPage";
@@ -18,8 +19,6 @@ import PageHeader from "../PageHeader";
 import CommunityPageSubHeader from "./CommunityPageSubHeader";
 import InfoPageSection from "./InfoPageSection";
 
-const isPublicTripsEnabled = process.env.NODE_ENV !== "production";
-
 const StyledTitle = styled(Typography)(() => ({
   marginTop: theme.spacing(3),
 }));
@@ -34,6 +33,7 @@ export default function CommunityPage({
   edit: boolean | undefined;
 }) {
   const { t } = useTranslation([COMMUNITIES]);
+  const isPublicTripsEnabled = useFeatureValue("public_trips_enabled", false);
 
   return (
     <CommunityBase communityId={communityId}>

app/web/features/communities/CommunityPage/CommunityPageSubHeader.tsx

@@ -1,3 +1,4 @@
+import { useFeatureValue } from "@growthbook/growthbook-react";
 import { TabContext } from "@mui/lab";
 import { Breadcrumbs, styled, Typography } from "@mui/material";
 import BetaFlag from "components/BetaFlag";
@@ -13,8 +14,6 @@ import { CommunityTab, routeToCommunity } from "routes";
 
 import JoinCommunityButton from "./JoinCommunityButton";
 
-const isPublicTripsEnabled = process.env.NODE_ENV !== "production";
-
 const StyledBreadcrumbsContainer = styled("div")(() => ({
   display: "flex",
   alignItems: "center",
@@ -35,6 +34,7 @@ export default function CommunityPageSubHeader({
   tab: CommunityTab;
 }) {
   const { t } = useTranslation([COMMUNITIES, PUBLIC_TRIPS]);
+  const isPublicTripsEnabled = useFeatureValue("public_trips_enabled", false);
 
   const router = useRouter();
   const communityTabBarLabels: Partial<

app/web/features/dashboard/Dashboard.tsx

@@ -1,3 +1,4 @@
+import { useFeatureValue } from "@growthbook/growthbook-react";
 import { Alert, Box, Grid, Typography } from "@mui/material";
 import Divider from "components/Divider";
 import HtmlMeta from "components/HtmlMeta";
@@ -19,10 +20,9 @@ import MyEvents from "./MyEvents";
 import ReminderCarousel from "./ReminderCarousel";
 import UpcomingStays from "./UpcomingStays";
 
-const isPublicTripsEnabled = process.env.NEXT_PUBLIC_COUCHERS_ENV !== "prod";
-
 export default function Dashboard() {
   const { t } = useTranslation([GLOBAL, DASHBOARD]);
+  const isPublicTripsEnabled = useFeatureValue("public_trips_enabled", false);
 
   return (
     <>

app/web/pages/my-public-trips.tsx

@@ -1,3 +1,4 @@
+import { useFeatureValue } from "@growthbook/growthbook-react";
 import { appGetLayout } from "components/AppRoute";
 import MyPublicTripsPage from "features/publicTrips/MyPublicTripsPage";
 import {
@@ -10,10 +11,8 @@ import { GetStaticProps } from "next";
 import nextI18nextConfig from "next-i18next.config";
 import { serverSideTranslations } from "next-i18next/serverSideTranslations";
 
-const isPublicTripsEnabled = process.env.NEXT_PUBLIC_COUCHERS_ENV !== "prod";
-
 export const getStaticProps: GetStaticProps = async ({ locale }) => {
-  if (!isPublicTripsEnabled) {
+  if (process.env.NEXT_PUBLIC_COUCHERS_ENV === "prod") {
     return { notFound: true };
   }
   return {
@@ -28,6 +27,12 @@ export const getStaticProps: GetStaticProps = async ({ locale }) => {
 };
 
 export default function MyPublicTrips() {
+  const isPublicTripsEnabled = useFeatureValue("public_trips_enabled", false);
+
+  if (!isPublicTripsEnabled) {
+    return null;
+  }
+
   return <MyPublicTripsPage />;
 }