Skip to content

fix: copy over hooks from base schema#3

Open
vkarpov15 wants to merge 1 commit intodoasync:masterfrom
vkarpov15:patch-1
Open

fix: copy over hooks from base schema#3
vkarpov15 wants to merge 1 commit intodoasync:masterfrom
vkarpov15:patch-1

Conversation

@vkarpov15
Copy link
Copy Markdown

Hi,

Thanks for the elegant lib! You're missing support for a couple things though, like copying hooks from the base schema (Automattic/mongoose#5897) .

An alternative implementation would be to do:

function extendSchema(schema, definition) {
  var clone = schema.clone();
  schema.add(definition);
  return schema;
}

This will copy over hooks and everything else, like plugins, query methods, statics, etc. Any thoughts on that?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant