Azure / Azure/data-api-builder

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

Offen
#2,132 2 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @severussundar Auf GitHub ansehen
Multiple mutations refactor
Vorherrschende Sprache
C#
Sterne
1.5k
Forks
370
Ø Merge
3 T. 22 Std.
Gemergte PRs (30 T.)
9

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.