epam / epam/epam-graphql

Inconsistent order of OnWrite/Editable calls for regular fields and for batch fields

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
23
Forks
6
PR merge metrics
No merged PRs in 30d

Description

### Description

For regular fields:
```c#
Field(unit => unit.Name)
.Editable()
.OnWrite((ctx, entity, value) => { });
```
For batch fields:
```c#
Field("name")
.FromBatch(entities => entities.ToDictionary(entity => entity, entity => entity.Name))
.OnWrite((ctx, entity, value) => { })
.Editable();
```

### Describe the solution you'd like

For both regular and batch fields order of calls `OnWrite` and `Editable` should be the same.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.