Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Benjie <[email protected]>
  • Loading branch information
jcgsville and benjie authored Jan 3, 2025
1 parent c1e65a7 commit 8383d32
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions utils/website/graphile-config/preset.md
Original file line number Diff line number Diff line change
Expand Up @@ -194,15 +194,15 @@ below.

:::note Merging exists-but-undefined options

When extending presets, Graphile Config merges options scopes with
`Object.assign({}, baseScope, extendingScope)`. This has the following
implications:

1. When an option is set to `undefined` or `null`, it will overwrite the value
in the extended preset. Depending on the library behavior, this may result in
the library default being used for the option.
2. When an option is not set in a scope, the resolved value will be that of the
extended preset.
When extending presets, Graphile Config merges options scopes with logic
equivalent to `Object.assign({}, baseScope, extendingScope)`. This has the
following implications:

1. When an option is set to `undefined` or `null` in the extending preset, it
will override the value in the base preset. Depending on the library
behavior, this may result in the library default being used for the option.
2. When an option is _not present_ in the extending scope, the resolved value
will be that of the option in the base scope (if any).

:::

Expand Down

0 comments on commit 8383d32

Please sign in to comment.