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

[MVC구현 / 3단계] 로이(김덕우) 미션 제출합니다. #631

Merged
merged 5 commits into from
Sep 27, 2023

Conversation

the9kim
Copy link

@the9kim the9kim commented Sep 25, 2023

안녕하세요 망고!

미션 구현이 좀 늦었네요ㅠ

2단계에서 질문주신 내용에 제대로된 답변을 못드린 듯해서 부끄럽네요..

부족한 부분은 추후 좀 더 공부해보겠습니다.

시간 날 때 편하게 리뷰 주시면 감사하겠습니다!

Copy link

@Go-Jaecheol Go-Jaecheol left a comment

Choose a reason for hiding this comment

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

안녕하세요 로이!

MVC 미션 마무리하느라 고생하셨습니다.
전체적으로 봤을 때 수정할 부분이 보이지 않아서 머지하려고 했는데,
DispatcherServletInitializer의 패키지 위치 때문에 API가 제대로 동작하지 않는 것 같아요..!

이 부분만 수정해주세요 😎
(시간 남으신다면 한 번 이유도 고민해보면 좋을 것 같아요 😊)

Comment on lines +19 to +22
return this.handlerAdapters.stream()
.filter(handlerAdapter -> handlerAdapter.isSupport(handler))
.findFirst()
.orElseThrow(() -> new IllegalArgumentException("해당 handler를 찾을 수 없습니다."));

Choose a reason for hiding this comment

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

피드백 반영 좋아요 👍👍

Choose a reason for hiding this comment

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

지금 Controller가 제대로 동작하고 있는지 http://localhost:8080/login/view와 같은 url로 확인했을 때, 제대로 된 페이지 뷰가 나오지 않는 걸 확인했어요!
오늘 강의 초반에도 언급되었던 DispatcherServletInitializer의 패키지 위치 때문인 것 같아요!

@the9kim
Copy link
Author

the9kim commented Sep 27, 2023

안녕하세요 망고!

꼼꼼하게 챙기지 못했네요ㅠ

제가 놓친 부분 까지 살펴봐주셔서 감사합니다👍

말씀하신대로 DispatcherServletInitializer 패키지 위치가 문제였던 것 같습니다.

패키지 app으로 위치를 바꾸는 방식으로 수정해봤어요!

리뷰 감사했습니다.

명절 잘 보내세요!!

Copy link

@Go-Jaecheol Go-Jaecheol left a comment

Choose a reason for hiding this comment

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

MVC 미션 고생하셨습니다 로이!! 🎉

이번 미션은 여기서 마무리할게요

즐거운 추석 보내세요~~ 😎

@Go-Jaecheol Go-Jaecheol merged commit dd7d7de into woowacourse:the9kim Sep 27, 2023
1 check failed
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