fix: don't display setup page icon for these apps (#13537)

* fix: don't display setup page icon for these apps * chore: make it optional * chore: feedback --------- Co-authored-by: Keith Williams <keithwillcode@gmail.com>
calcom · Feb 6, 2024 · f09153b · f09153b
1 parent 3e492eb
commit f09153b
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 3 deletions.
diff --git a/packages/app-store/_components/AppCard.tsx b/packages/app-store/_components/AppCard.tsx
@@ -21,6 +21,7 @@ export default function AppCard({
   teamId,
   disableSwitch,
   switchTooltip,
+  hideSettingsIcon = false,
 }: {
   app: RouterOutputs["viewer"]["integrations"]["items"][number] & { credentialOwner?: CredentialOwner };
   description?: React.ReactNode;
@@ -32,6 +33,7 @@ export default function AppCard({
   LockedIcon?: React.ReactNode;
   disableSwitch?: boolean;
   switchTooltip?: string;
+  hideSettingsIcon?: boolean;
 }) {
   const { t } = useTranslation();
   const [animationRef] = useAutoAnimate<HTMLDivElement>();
@@ -124,9 +126,11 @@ export default function AppCard({
         {app?.isInstalled && switchChecked ? (
           app.isSetupAlready === undefined || app.isSetupAlready ? (
             <div className="relative p-4 pt-5 text-sm [&_input]:mb-0 [&_input]:leading-4">
-              <Link href={`/apps/${app.slug}/setup`} className="absolute right-4 top-4">
-                <Settings className="text-default h-4 w-4" aria-hidden="true" />
-              </Link>
+              {!hideSettingsIcon && (
+                <Link href={`/apps/${app.slug}/setup`} className="absolute right-4 top-4">
+                  <Settings className="text-default h-4 w-4" aria-hidden="true" />
+                </Link>
+              )}
               {children}
             </div>
           ) : (

diff --git a/packages/app-store/fathom/components/EventTypeAppCardInterface.tsx b/packages/app-store/fathom/components/EventTypeAppCardInterface.tsx
@@ -13,6 +13,7 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
 
   return (
     <AppCard
+      hideSettingsIcon
       app={app}
       switchOnClick={(e) => {
         updateEnabled(e);

diff --git a/packages/app-store/ga4/components/EventTypeAppCardInterface.tsx b/packages/app-store/ga4/components/EventTypeAppCardInterface.tsx
@@ -13,6 +13,7 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
 
   return (
     <AppCard
+      hideSettingsIcon
       app={app}
       switchOnClick={(e) => {
         updateEnabled(e);

diff --git a/packages/app-store/gtm/components/EventTypeAppCardInterface.tsx b/packages/app-store/gtm/components/EventTypeAppCardInterface.tsx
@@ -13,6 +13,7 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
 
   return (
     <AppCard
+      hideSettingsIcon
       app={app}
       switchOnClick={(e) => {
         updateEnabled(e);

diff --git a/packages/app-store/matomo/components/EventTypeAppCardInterface.tsx b/packages/app-store/matomo/components/EventTypeAppCardInterface.tsx
@@ -14,6 +14,7 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
 
   return (
     <AppCard
+      hideSettingsIcon
       app={app}
       switchOnClick={(e) => {
         updateEnabled(e);

diff --git a/packages/app-store/metapixel/components/EventTypeAppCardInterface.tsx b/packages/app-store/metapixel/components/EventTypeAppCardInterface.tsx
@@ -13,6 +13,7 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
 
   return (
     <AppCard
+      hideSettingsIcon
       app={app}
       switchOnClick={updateEnabled}
       switchChecked={enabled}

diff --git a/packages/app-store/plausible/components/EventTypeAppCardInterface.tsx b/packages/app-store/plausible/components/EventTypeAppCardInterface.tsx
@@ -14,6 +14,7 @@ const EventTypeAppCard: EventTypeAppCardComponent = function EventTypeAppCard({
 
   return (
     <AppCard
+      hideSettingsIcon
       app={app}
       switchOnClick={(e) => {
         updateEnabled(e);