Merge branch 'master' into azaslavsky/backup/test_coverage

getsentry · Jul 20, 2023 · 22f26b5 · 22f26b5
2 parents d8740f8 + e2f0024
commit 22f26b5
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/static/app/views/settings/organizationIntegrations/integrationDetailedView.tsx b/static/app/views/settings/organizationIntegrations/integrationDetailedView.tsx
@@ -348,7 +348,7 @@ class IntegrationDetailedView extends AbstractIntegrationDetailedView<
             help: t(
               'Allow Sentry to comment on pull requests about issues impacting your app.'
             ),
-            disabled: !hasIntegration || !hasOrgWrite,
+            disabled: !hasIntegration,
             disabledReason: t(
               'You must have a GitHub integration to enable this feature.'
             ),
@@ -370,7 +370,11 @@ class IntegrationDetailedView extends AbstractIntegrationDetailedView<
         initialData={initialData}
         onSubmitError={() => addErrorMessage('Unable to save change')}
       >
-        <JsonForm features={organization.features} forms={forms} />
+        <JsonForm
+          disabled={!hasOrgWrite}
+          features={organization.features}
+          forms={forms}
+        />
       </Form>
     );
   }