99designs / 99designs/gqlgen

omit_resolver_fields option does not respect `models.<model>.fields.<field>.resolver` option

未关闭
#3,044 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Go
星标
10.8k
派生
1.3k
平均合并
2 天 36 分钟
30 天内合并 PR
26

描述

### What happened?

The `omit_resolver_fields` option that introduced by https://github.com/99designs/gqlgen/pull/2957 does not respect `models..fields..resolver` option, only schema directives are respected.

#### gqlgen.yml

```testdata/gqlgen_omit_resolver_fields_with_explicit_resolver.yml
# testdata/gqlgen_omit_resolver_fields_with_explicit_resolver.yml
schema:
- "schema_omit_resolver_fields_with_explicit_resolver.graphql"

exec:
filename: out_omit_resolver_fields/ignored.go
model:
filename: out_omit_resolver_fields/generated.go

omit_resolver_fields: true
models:
Base:
fields:
ResolverField:
resolver: true
```

#### schema.graphql

```testdata/schema_omit_resolver_fields_with_explicit_resolver.graphql
# testdata/schema_omit_resolver_fields_with_explicit_resolver.graphql
type Base {
StandardField: String!
ResolverField: String!
}
```

### What did you expect?

The `models..fields..resolver` option marks the field to be resolved with explit resolvers, so the `omit_resolver_fields` should respect also this option.

### Minimal graphql.schema and models to reproduce

```schema_omit_resolver_fields_with_explicit_resolver.graphql
type Base {
StandardField: String!
ResolverField: String!
}
```

### versions
- `go run github.com/99designs/gqlgen version`?
- `v0.17.45-dev`
- `go version`?
- `go version go1.22.0 linux/amd64`

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。