moleculer-db icon indicating copy to clipboard operation
moleculer-db copied to clipboard

imp(core): Modify the entry value of the default setting to undefined…

Open 0x0a0d opened this issue 2 years ago • 2 comments

Limitation in the mergeSchemaSettings function (service.js#L668), where the default moleculer-db setting's entry value was set to null, preventing it from being overridden by other mixins.

By changing the default entry value to undefined, it now aligns with Lodash’s defaultsDeep method, which prioritizes values that are not undefined

0x0a0d avatar Oct 25 '23 05:10 0x0a0d

You can overwrite these values as well, just you should reorder the mixins

icebob avatar Nov 12 '23 16:11 icebob

I think it is better to change null to undefined. You can bump the minor version

0x0a0d avatar Nov 12 '23 16:11 0x0a0d