[dagit] Add dummy graphName field to AssetNode (#7637)

## Summary Add a `graphName` field to `AssetNode`. Just returns the first op name for now. ## Test Plan Load Dagit, view an asset, verify that `graphName` has been populated in the query response.
dagster-io · Apr 28, 2022 · 5870039 · 5870039
1 parent 8ab7bfb
commit 5870039
Show file tree

Hide file tree

Showing 7 changed files with 18 additions and 0 deletions.
diff --git a/js_modules/dagit/packages/core/src/asset-graph/AssetNode.tsx b/js_modules/dagit/packages/core/src/asset-graph/AssetNode.tsx
@@ -218,6 +218,7 @@ export const ASSET_NODE_LIVE_FRAGMENT = gql`
 export const ASSET_NODE_FRAGMENT = gql`
   fragment AssetNodeFragment on AssetNode {
     id
+    graphName
     opName
     opNames
     description

diff --git a/js_modules/dagit/packages/core/src/asset-graph/types/AssetGraphQuery.ts b/js_modules/dagit/packages/core/src/asset-graph/types/AssetGraphQuery.ts
diff --git a/js_modules/dagit/packages/core/src/asset-graph/types/AssetNodeFragment.ts b/js_modules/dagit/packages/core/src/asset-graph/types/AssetNodeFragment.ts
diff --git a/js_modules/dagit/packages/core/src/assets/types/AssetNodeDefinitionFragment.ts b/js_modules/dagit/packages/core/src/assets/types/AssetNodeDefinitionFragment.ts
diff --git a/js_modules/dagit/packages/core/src/assets/types/AssetQuery.ts b/js_modules/dagit/packages/core/src/assets/types/AssetQuery.ts
diff --git a/js_modules/dagit/packages/core/src/graphql/schema.graphql b/js_modules/dagit/packages/core/src/graphql/schema.graphql
diff --git a/python_modules/dagster-graphql/dagster_graphql/schema/asset_graph.py b/python_modules/dagster-graphql/dagster_graphql/schema/asset_graph.py
@@ -90,6 +90,7 @@ class GrapheneAssetNode(graphene.ObjectType):
     dependencies = non_null_list(GrapheneAssetDependency)
     dependencyKeys = non_null_list(GrapheneAssetKey)
     description = graphene.String()
+    graphName = graphene.String()
     id = graphene.NonNull(graphene.ID)
     jobNames = non_null_list(graphene.String)
     jobs = non_null_list(GraphenePipeline)
@@ -294,6 +295,13 @@ def resolve_dependencies(self, graphene_info) -> List[GrapheneAssetDependency]:
             for dep in self._external_asset_node.dependencies
         ]
 
+    def resolve_graphName(self, _graphene_info) -> Optional[str]:
+        # todo OwenKephart - return the correct graph name here
+        if self._external_asset_node.op_name:
+            return self._external_asset_node.op_name
+        else:
+            return None
+
     def resolve_jobNames(self, _graphene_info) -> List[str]:
         return self._external_asset_node.job_names