KDT5_안태욱 2차과제 제출(re) #82
Conversation
There was a problem hiding this comment.
태욱님 밀린 강의 들으시느라 고생이 많으십니다👍🏻 그와중에 과제까지 제출하신점 칭찬드립니다. 다만, 현재 footer 컴포넌트가 선언되지 않았고, 영화 상세정보 페이지 또한 보여지지 않고 있습니다. 꼭 강의듣고 난 이후에 정상적으로 페이지가 보여질 수 있도록 확인 부탁드립니다:)
++ 단순히 강의의 내용을 따라하는 과정이지만 꼭 추후에는 리팩토링을 통해 자신의 영역으로 재구현하시길 바랍니다.
추가적으로 생각해볼만한 부분 코멘트 드렸습니다. 확인해보시고 리팩토링시에 꼭 적용해주세요!
| } | ||
| export const getMovieDetails = async id => { | ||
| try { | ||
| const res = await fetch(`http://omdbapi.com?apikey=7035c60c&i=${id}&plot=full`) |
There was a problem hiding this comment.
현재 배포하신 서버는 https 프로토콜을 사용하고 있습니다. https는 HTTP의 보안된 통신을 수행하기 때문에 http에 대한 요청은 불가능합니다. https로 바꾸셔서 진행하시면 영화 상세정보 또한 확인하실 수 있습니다:)
| import NotFound from './NotFoundPage' | ||
|
|
||
| export default createRouter([ | ||
| { path: '#/', component: Home }, |
There was a problem hiding this comment.
현재 모든 페이지가 #을 포함하고 있습니다. movie라는 단어가 해당 페이지에 대한 의미를 내포하듯, URL에서 모든 문자는 의미를 가져야하는데요. #는 어떠한 의미를 가지고 있는지 설명해주실 수 있을까요?
| class="photo"></div> | ||
| <p class="name">${name}</p> | ||
| <p> | ||
| <a href="https://mail.google.com/mail/?view=cm&fs=1&to=${email}" target="_blank"> |
| <div class="the-loader hide"></div> | ||
| ` | ||
|
|
||
| const moviesEl = this.el.querySelector('.movies') |
There was a problem hiding this comment.
영화 목록 태그를 클래스명으로 조회하고 있는데, CSS 특이성에 따르면 구현하신 페이지 내에서 MovieList 컴포넌트는 하나만 사용되므로 클래스명이 아닌 ID를 통해서 요소를 조회하는 것이 더 효율적일 것으로 보입니다. 이와 마찬가지로 아래의 loadEl 또한 ID를 통해서 조회하는 방식을 검토할 수 있을 것 같아요! 클래스명을 사용하여 조회해야한다면 그 이유도 설명해주실 수 있을까요?🤔
| render() { | ||
| const theHeader = new TheHeader().el | ||
| const routerView = document.createElement('router-view') | ||
| const theFooter = document.createElement('router-view') |
There was a problem hiding this comment.
routerView 컴포넌트와 theFooter 컴포넌트가 동일한 의미를 담고 있는데, 오타로 보입니다:)
|
현재는 단 하나의 커밋으로 모든 변경사항을 반영하고 계신데요. commit 메시지 또한 기능별로 구분하여 커밋을 진행해보는 것을 추천드립니다. 기능별로 구분하는 경우에 추후 백업이 필요한 시점에 각 기능별로 반영할 수 있으며, 메시지만으로도 충분히 현재 태욱님의 프로젝트의 과정을 파악할 수 있기 때문입니다:) |
배포사이트 : https://moviesiteprojectbyahn.netlify.app/#/
❗ 필수
❔ 선택