Skip to content

1회차 숙제#5

Open
nayoung04 wants to merge 2 commits intoChuseok22:mainfrom
nayoung04:feature/nayoung
Open

1회차 숙제#5
nayoung04 wants to merge 2 commits intoChuseok22:mainfrom
nayoung04:feature/nayoung

Conversation

@nayoung04
Copy link
Copy Markdown

No description provided.

return userService.saveUser(username, password, nickname);
}

@GetMapping("/users/id/{id}")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

username으로 찾는건데 id로 찾은 이유가 궁금합니다

this.userService = userService;
}

@PostMapping("/users/join")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

아 requestparam으로 하는 방법도 있군요! 전 그냥 하드 코딩했는데...

@Chuseok22
Copy link
Copy Markdown
Owner

Good!!
comment해줄 부분이 없네 ^^b

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.

3 participants