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

Add docs for DiscoveryService.createDecorator #2876

Open
micalevisk opened this issue Oct 8, 2023 · 2 comments
Open

Add docs for DiscoveryService.createDecorator #2876

micalevisk opened this issue Oct 8, 2023 · 2 comments

Comments

@micalevisk
Copy link
Member

micalevisk commented Oct 8, 2023

there's no mention about DiscoveryService built-in provider from @nestjs/core

so there's no mention about the feature add in nestjs v10.2.0 nestjs/nest#12107

I'd like to see the DiscoveryService.createDecorator somewhere in the docs like we have for Reflector.createDecorator. Not sure where, tho 🤔

This is to clarify mainly this:

  1. we can't use the same decorator more than once for each target class because only the first (top-down order) one will be taken into account
@andreasvh-conceto
Copy link

Is there something in progress? It would be absolutely great to have this documented.

Thx :)

@micalevisk
Copy link
Member Author

micalevisk commented Jun 5, 2024

@andreasvh-conceto I won't work on this soon

Feel free to open a PR to address this

@micalevisk micalevisk moved this to 🚧 Working on this! in My contributions to NestJS framework 😻 Jul 2, 2024
@micalevisk micalevisk moved this from 🚧 Working on this! to 💭 Maybe I'll take this in My contributions to NestJS framework 😻 Sep 21, 2024
@micalevisk micalevisk moved this from 💭 Maybe I'll take this to 🕒 I'm trying... in My contributions to NestJS framework 😻 Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants