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

feat(crud-typeorm): ignore entity decorators when saving #692

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat(crud-typeorm): ignore entity decorators when saving #692

wants to merge 1 commit into from

Conversation

Char2sGu
Copy link

@Char2sGu Char2sGu commented Apr 7, 2021

The @Exclude() decorator is used to exclude some fields in the entity
in responses, but will cause confusion when creating entities because
the fields will also be excluded before saving to the database.

@rewiko
Copy link
Collaborator

rewiko commented Apr 7, 2021

Nice, would you be able to add a test to check this behaviour?

The `@Exclude()` decorator is used to exclude some fields in the entity
in responses, but will cause confusion when creating entities because
the fields will also be excluded before saving to the database.
@Char2sGu
Copy link
Author

Char2sGu commented Apr 8, 2021

Nice, would you be able to add a test to check this behaviour?

Sorry, but I am really new to testing and don’t know how to do that. :-(

@rewiko
Copy link
Collaborator

rewiko commented Apr 8, 2021

Sorry, but I am really new to testing and don’t know how to do that. :-(

No problem I can add the test.

@rewiko rewiko mentioned this pull request Nov 28, 2021
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants