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 #2

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Feat/crud #2

wants to merge 13 commits into from

Conversation

steadfastree
Copy link
Owner

No description provided.

user: User;

@ApiProperty({ description: '작성 유저 고유 번호' })
@IsNumber()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Entity에는 검증 로직, swagger decorator를 붙이지 않는게 조아보여

);*/

const user = await this.userRepository.findOneOrFail({
// 여기서 email에 undefined가 들어가면 테이블의 첫 번째 값을 출력하는데 왜인지 모르겠음.. 일단 유효성 검사로 일어나지 않게 함.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typeorm/typeorm#2500
이런게 잇는 듯

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

머야 결국 안고쳐진건가 ㅋㅋ

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.

None yet

2 participants