Azure / Azure/data-api-builder

Evaluate if refactor of CreateMutation.Build() to accept RuntimeConfig object is beneficial

Aperta
#2,132 2 commenti 0 reazioni 1 assegnatario Rivendicata da @severussundar Vedi su GitHub
Multiple mutations refactor
Lingua principale
C#
Stelle
1.5k
Fork
370
Merge medio
3g 17h
PR unite (30g)
8

Descrizione

PR https://github.com/Azure/data-api-builder/pull/2116 introduces a new parameter called `isMultipleCreateOperationEnabled` to `CreateMutationBuilder.Build()` method.

`isMultipleCreateOperationEnabled` can be computed using `RuntimeConfig.IsMultipleCreateOperationEnabled()`.
`CreateMutationBuilder.Build()` method accepts another parameter called `entities` of type `RuntimeEntities`. `entities` originates from `RuntimeConfig.RuntimeEntities`

Evaluate if it is beneficial to refactor `CreateMutationBuilder.Build()` method to accept `RuntimeConfig` object instead of two parameters separately.

Original discussion: https://github.com/Azure/data-api-builder/pull/2116#discussion_r1536631102

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.