-
Notifications
You must be signed in to change notification settings - Fork 87
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
Replace custom instance creation with class-transformer #1327
Comments
Hey I have worked on around 7-8 company based project, but never countribute to open source, this is my first time can you assign me this task. |
I wanna give this issue a try , can you please assign this issue to me @NickSeagull |
Woops, got lost in the notifications. Anyone available? I'll assign to the first one who replies 🙏 |
@NickSeagull with my limited knowledge , I understand that using the class-transformer it will simplify the process of creating class instances, especially when dealing with complex scenarios like deep cloning. But not stated clearly whether the changes have to be made in whole source code or a specific file ? |
@NickSeagull i am new to opensource. Can i give it a try? |
.take |
Currently we're creating a lot of class instances in the core using our own functions that do deep cloning, etc.
This is very brittle as we're not sure that we are covering all the edge cases of class instance creation.
Libraries like class-transformer are specialized on this and make the life easier
The text was updated successfully, but these errors were encountered: