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
Decorator return type has to be any or would have to provide 2 decorators, one each returning PropertyDecorator/ClassDecorator(old decorator type) and ESClassFieldDecorator/ESClassDecorator(new decorator type, manual); otherwise typescript errors with Unable to resolve signature of property decorator when called as an expression. for property decorators, see @prop as a example
typescript does not output decorator type metadata for the new ES Decorators
a good way to test without having to completely copy the whole test-suite
default classes which use decorators cannot be provided anymore because decorator settings may be different than the typegoose project and then be incompatible
typescript 5.2 requirement (and waiting for this version to be released)
PS: the branch feature/try-new-decorators is just a testing implementation, it is not meant to actually be merged in its current state (its using WIP commits and the tests are not fixed)
The text was updated successfully, but these errors were encountered:
Describe what you need | want
Typescript 5.0 has added support for ES Decorators and Typescript 5.2 has added support for ES Decorator metadata
Do you have already an idea for the implementation?
yes, see branch
feature/try-new-decorators
full implementation currently blocked by:
any
or would have to provide 2 decorators, one each returningPropertyDecorator
/ClassDecorator
(old decorator type) andESClassFieldDecorator
/ESClassDecorator
(new decorator type, manual); otherwise typescript errors withUnable to resolve signature of property decorator when called as an expression.
for property decorators, see@prop
as a examplereflect-metadata
(or any other) will not be installed / run by default anymore, fixes Allow alternatives to reflect-metadata for shimming reflect metadata API #761PS: the branch
feature/try-new-decorators
is just a testing implementation, it is not meant to actually be merged in its current state (its using WIP commits and the tests are not fixed)The text was updated successfully, but these errors were encountered: