Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish new release to fix nestjs InjectMapper decorator #542

Open
ttshivers opened this issue Apr 9, 2023 · 1 comment
Open

Publish new release to fix nestjs InjectMapper decorator #542

ttshivers opened this issue Apr 9, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@ttshivers
Copy link

Is your feature request related to a problem? Please describe.

Can a new release be published to fix the nestjs InjectMapper decorator type.
This blocks nestjs projects from updating to typescript 5 as it now has tighter parameter decorator checking.
microsoft/TypeScript#52435

It was fixed in nestjs in nestjs/nest#10970. The main branch has already updated nestjs to a version with this but a release hasn't been made since then.

Describe the solution you'd like

Can a new release be published to fix the nestjs InjectMapper decorator type.

Describe alternatives you've considered

I could write my own wrapper decorator to fix the typing but having a new release would be ideal.

Additional context

No response

@ttshivers ttshivers added the enhancement New feature or request label Apr 9, 2023
@tadavid-cae
Copy link
Contributor

@ttshivers temp solution: change @InjectMapper() to @Inject(getMapperToken())

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants