Skip to content

Commit

Permalink
remove executions resolver on GrapheneAssetCheck
Browse files Browse the repository at this point in the history
  • Loading branch information
johannkm committed Nov 2, 2023
1 parent cdccbb8 commit 3a984c1
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 28 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 0 additions & 7 deletions js_modules/dagster-ui/packages/ui-core/src/graphql/types.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -132,12 +132,6 @@ class GrapheneAssetCheck(graphene.ObjectType):
name = graphene.NonNull(graphene.String)
assetKey = graphene.NonNull(GrapheneAssetKey)
description = graphene.String()
# deprecated. Use the top level assetCheckExecutions resolver instead
executions = graphene.Field(
non_null_list(GrapheneAssetCheckExecution),
limit=graphene.NonNull(graphene.Int),
cursor=graphene.String(),
)
executionForLatestMaterialization = graphene.Field(GrapheneAssetCheckExecution)
canExecuteIndividually = graphene.NonNull(GrapheneAssetCheckCanExecuteIndividually)

Expand All @@ -163,20 +157,6 @@ def resolve_name(self, _) -> str:
def resolve_description(self, _) -> Optional[str]:
return self._asset_check.description

def resolve_executions(
self, graphene_info: ResolveInfo, **kwargs
) -> List[GrapheneAssetCheckExecution]:
from dagster_graphql.implementation.fetch_asset_checks import (
fetch_asset_check_executions,
)

return fetch_asset_check_executions(
graphene_info.context.instance,
self._asset_check.key,
kwargs["limit"],
kwargs.get("cursor"),
)

def resolve_executionForLatestMaterialization(
self, _graphene_info: ResolveInfo
) -> Optional[GrapheneAssetCheckExecution]:
Expand Down

0 comments on commit 3a984c1

Please sign in to comment.