-
Notifications
You must be signed in to change notification settings - Fork 304
[MVC 구현 - 3단계] 홍실(홍혁준) 미션 제출합니다. #620
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안녕하세요 홍실 ! 고생하셨슴다 !
너무 잘 구현해주셔서 리뷰할게 좀 적네요 ...
궁금한게 있어서 몇 개 달아놨습니다 !
다음 번에는 머지해도 될 것 같습니다 ~
...ebmvc/org/springframework/web/servlet/mvc/tobe/handler_mapping/AnnotationHandlerMapping.java
Outdated
Show resolved
Hide resolved
mvc/src/main/java/webmvc/org/springframework/web/servlet/view/JsonView.java
Outdated
Show resolved
Hide resolved
if (model.size() == 1) { | ||
return objectMapper.writeValueAsString(model.values().toArray()[0]); | ||
} | ||
return objectMapper.writeValueAsString(model); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@DisplayName("JspView render Test 값이 두개 이상일 떄") | ||
void renderTest2() throws Exception { | ||
final JsonView jsonView = new JsonView(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트 꼼꼼 굿이에요...!
SonarCloud Quality Gate failed.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
홍실 고생하셨습니다 ㅎㅎ
MVC미션에서 함께해서 재밌었습니다 ^_^
하디 안녕하세요
이번 3단계 미션에서는 JSON, JSP View를 구현했습니다.
그리고 레거시 MVC도 제거했습니다.
이번 리뷰도 잘 부탁드립니다.(늦게 보내드려 죄송하네요)
가감없이 리뷰해주시면 감사하겠습니다.