-
Notifications
You must be signed in to change notification settings - Fork 823
feat: add decorator name to validation metadata #1687
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
feat: add decorator name to validation metadata #1687
Conversation
@Prim3591 thanks for approving, can you approve the workflow run as well please? |
@Prim3591 Bumping this, I still have an ongoing project that is blocked by this change... |
Thank you for your enthusiasm but I'm not an admin on this repo, and thus not able to merge it. I've just reviewed it because I need it too, by doing so I was expecting the "real admin" could perform this task quicker :) |
2ef8ff0
to
0da8f89
Compare
Can you rebase this please to the latest |
Done! |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
Added the optional
name
property toValidationMetadata
which is set fromoptions
inregisterDecorator
.Previously, the information in
options.name
was lost - it was not possible to find out which concrete decorators are present in a class.See #622.
Checklist
Update index.md
)develop
)npm run prettier:check
passesnpm run lint:check
passesFixes
references #622