-
Notifications
You must be signed in to change notification settings - Fork 118
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
Custom resolver won't load into i18n module #403
Comments
@BobbiSixkiller can you send a small repo containing the issue? A zip is enough :) |
First of all thanks a lot @toonvanstrijp for such a fast reply:) here's the link. Basically the idea is that the whole nestjs app consists of mail module and rmq module and responds to an incoming message from rmq by sending email. The email itself has to be localized and I wanted to use the custom resolver to resolve proper locale that comes in with the message object. |
Having same issue, temporary downgraded to |
@BobbiSixkiller sorry for the late reply, do you still need help with this issue? |
@toonvanstrijp yes, I even tried to downgrade the version as mentioned by @Tomas2D but still no luck. Right now I set the locale when sending the mail via MailerService sendmail method's context property. Resolving the locale inside custom resolver based on incoming RMQ message would be way more ellegant in my opion. Thank's again for coming back to me btw. |
@BobbiSixkiller feel free to make a PR to support this. I've not worked with NestJS and RMQ before. So any help is welcome! |
This change gives us a chance to implement any custom resolver we can. eg: toonvanstrijp#403
@BobbiSixkiller, tried to reproduce, just took sample/simple project with [email protected] and got correct logs and response. If you could, it will be better to have attached package-lock, Interesting in nestjs packages' versions and nestjs-i18n |
example of custom resolver for rabbitmq
and app.module.ts
|
Hi I'd like to reopen this issue because with current version "^9.2.1" I can not successfuly import custom Resolver. I'm testing this the same way as OP. It doesn't run the console log in resolve method so I guess the resolver does not load to i18n service at all.
custom.resolver.ts
email.module.ts
thanks for any help in advance!
The text was updated successfully, but these errors were encountered: