From b7b4f1c9a57d8b24533fbb65bb672cedc85caebb Mon Sep 17 00:00:00 2001
From: v-tarasevich-blitz-brain <v.tarasevich@blitz-brain.com>
Date: Wed, 8 Oct 2025 18:05:34 +0300
Subject: [PATCH] feat(uplodaFiles): add feature flag (#14951)

---
 .../datahub/graphql/resolvers/config/AppConfigResolver.java  | 1 +
 datahub-graphql-core/src/main/resources/app.graphql          | 5 +++++
 .../graphql/resolvers/config/AppConfigResolverTest.java      | 1 +
 datahub-web-react/src/appConfigContext.tsx                   | 1 +
 datahub-web-react/src/graphql/app.graphql                    | 1 +
 .../collectors/PropertiesCollectorConfigurationTest.java     | 1 +
 .../linkedin/datahub/graphql/featureflags/FeatureFlags.java  | 1 +
 .../configuration/src/main/resources/application.yaml        | 1 +
 8 files changed, 12 insertions(+)

diff --git a/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolver.java b/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolver.java
index deea5e4af4..8facdc7eda 100644
--- a/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolver.java
+++ b/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolver.java
@@ -266,6 +266,7 @@ public class AppConfigResolver implements DataFetcher<CompletableFuture<AppConfi
             .setLogicalModelsEnabled(_featureFlags.isLogicalModelsEnabled())
             .setShowHomepageUserRole(_featureFlags.isShowHomepageUserRole())
             .setAssetSummaryPageV1(_featureFlags.isAssetSummaryPageV1())
+            .setDocumentationFileUploadV1(_featureFlags.isDocumentationFileUploadV1())
             .build();
 
     appConfig.setFeatureFlags(featureFlagsConfig);
diff --git a/datahub-graphql-core/src/main/resources/app.graphql b/datahub-graphql-core/src/main/resources/app.graphql
index 72053effe5..ec7a3c9f49 100644
--- a/datahub-graphql-core/src/main/resources/app.graphql
+++ b/datahub-graphql-core/src/main/resources/app.graphql
@@ -802,6 +802,11 @@ type FeatureFlagsConfig {
   Enables displaying the asset summary page
   """
   assetSummaryPageV1: Boolean!
+
+  """
+  If enabled, allows uploading of files for documentation.
+  """
+  documentationFileUploadV1: Boolean!
 }
 
 """
diff --git a/datahub-graphql-core/src/test/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolverTest.java b/datahub-graphql-core/src/test/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolverTest.java
index 2765be140b..b21a174450 100644
--- a/datahub-graphql-core/src/test/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolverTest.java
+++ b/datahub-graphql-core/src/test/java/com/linkedin/datahub/graphql/resolvers/config/AppConfigResolverTest.java
@@ -132,6 +132,7 @@ public class AppConfigResolverTest {
     when(mockFeatureFlags.isLogicalModelsEnabled()).thenReturn(false);
     when(mockFeatureFlags.isShowHomepageUserRole()).thenReturn(false);
     when(mockFeatureFlags.isAssetSummaryPageV1()).thenReturn(false);
+    when(mockFeatureFlags.isDocumentationFileUploadV1()).thenReturn(false);
   }
 
   @Test
diff --git a/datahub-web-react/src/appConfigContext.tsx b/datahub-web-react/src/appConfigContext.tsx
index 599d137a07..42b1b25c6e 100644
--- a/datahub-web-react/src/appConfigContext.tsx
+++ b/datahub-web-react/src/appConfigContext.tsx
@@ -91,6 +91,7 @@ export const DEFAULT_APP_CONFIG = {
         logicalModelsEnabled: false,
         showHomepageUserRole: false,
         assetSummaryPageV1: false,
+        documentationFileUploadV1: false,
     },
     chromeExtensionConfig: {
         enabled: false,
diff --git a/datahub-web-react/src/graphql/app.graphql b/datahub-web-react/src/graphql/app.graphql
index f3b5fcbd79..0441a33c60 100644
--- a/datahub-web-react/src/graphql/app.graphql
+++ b/datahub-web-react/src/graphql/app.graphql
@@ -113,6 +113,7 @@ query appConfig {
             logicalModelsEnabled
             showHomepageUserRole
             assetSummaryPageV1
+            documentationFileUploadV1
         }
         chromeExtensionConfig {
             enabled
diff --git a/metadata-io/src/test/java/com/linkedin/metadata/system_info/collectors/PropertiesCollectorConfigurationTest.java b/metadata-io/src/test/java/com/linkedin/metadata/system_info/collectors/PropertiesCollectorConfigurationTest.java
index 997c4d0c4b..73be1f977c 100644
--- a/metadata-io/src/test/java/com/linkedin/metadata/system_info/collectors/PropertiesCollectorConfigurationTest.java
+++ b/metadata-io/src/test/java/com/linkedin/metadata/system_info/collectors/PropertiesCollectorConfigurationTest.java
@@ -464,6 +464,7 @@ public class PropertiesCollectorConfigurationTest extends AbstractTestNGSpringCo
           "featureFlags.businessAttributeEntityEnabled",
           "featureFlags.cdcModeChangeLog",
           "featureFlags.dataContractsEnabled",
+          "featureFlags.documentationFileUploadV1",
           "featureFlags.editableDatasetNameEnabled",
           "featureFlags.entityVersioning",
           "featureFlags.erModelRelationshipFeatureEnabled",
diff --git a/metadata-service/configuration/src/main/java/com/linkedin/datahub/graphql/featureflags/FeatureFlags.java b/metadata-service/configuration/src/main/java/com/linkedin/datahub/graphql/featureflags/FeatureFlags.java
index d62732fd75..27671e30a7 100644
--- a/metadata-service/configuration/src/main/java/com/linkedin/datahub/graphql/featureflags/FeatureFlags.java
+++ b/metadata-service/configuration/src/main/java/com/linkedin/datahub/graphql/featureflags/FeatureFlags.java
@@ -50,4 +50,5 @@ public class FeatureFlags {
   private boolean showHomepageUserRole = false;
   private boolean assetSummaryPageV1 = false;
   private boolean showDefaultExternalLinks = true;
+  private boolean documentationFileUploadV1 = false;
 }
diff --git a/metadata-service/configuration/src/main/resources/application.yaml b/metadata-service/configuration/src/main/resources/application.yaml
index 9e3601c413..c81aebec8e 100644
--- a/metadata-service/configuration/src/main/resources/application.yaml
+++ b/metadata-service/configuration/src/main/resources/application.yaml
@@ -842,6 +842,7 @@ featureFlags:
   fineGrainedLineageNotAllowedForPlatforms: ${FINE_GRAINED_LINEAGE_NOT_ALLOWED_FOR_PLATFORMS:} # Comma separated list of platforms for which schemaFields entity edges will not be allowed to be created. for example: "hdfs, s3"
   assetSummaryPageV1: ${ASSET_SUMMARY_PAGE_V1:false} # Enables displaying the asset summary page
   showDefaultExternalLinks: ${SHOW_DEFAULT_EXTERNAL_LINKS:true} # If turned on, show the default external links on the entity page
+  documentationFileUploadV1: ${DOCUMENTATION_FILE_UPLOAD_V1:false} # Enables uploading of files for documentation
 
 entityChangeEvents:
   enabled: ${ENABLE_ENTITY_CHANGE_EVENTS_HOOK:true}