Skip to content

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

Open
Char2sGu wants to merge 1 commit into
nestjsx:masterfrom
Char2sGu:master
Open

feat(crud-typeorm): ignore entity decorators when saving#692
Char2sGu wants to merge 1 commit into
nestjsx:masterfrom
Char2sGu:master

Conversation

@Char2sGu

@Char2sGu Char2sGu commented Apr 7, 2021

Copy link
Copy Markdown

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

rewiko commented Apr 7, 2021

Copy link
Copy Markdown
Collaborator

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

Char2sGu commented Apr 8, 2021

Copy link
Copy Markdown
Author

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

rewiko commented Apr 8, 2021

Copy link
Copy Markdown
Collaborator

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