You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It appears the current typings (tested at 09b3a90) are missing the include: [{all: true}] option on the Includable type in FindOptions for eager-loading all included associations (as described here).
I guess this could easily be fixed by e.g. adding all?: boolean; to the IncludeOptions interface. If that doesn't work for you due to problems with interface reuse, a new IncludeAllOptions interface would be minimal as well.
Happy to provide a PR if you want one for what could be a one-line change 😄
The text was updated successfully, but these errors were encountered:
On a completely unrelated note: I've also just now noticed that the QueryInterface typings are not directly exported from sequelize.d.ts. Was that on purpose/should they be accessed differently or was this an oversight?
It appears the current typings (tested at 09b3a90) are missing the
include: [{all: true}]
option on theIncludable
type inFindOptions
for eager-loading all included associations (as described here).I guess this could easily be fixed by e.g. adding
all?: boolean;
to theIncludeOptions
interface. If that doesn't work for you due to problems with interface reuse, a newIncludeAllOptions
interface would be minimal as well.Happy to provide a PR if you want one for what could be a one-line change 😄
The text was updated successfully, but these errors were encountered: