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

6-1 레스토랑 상세 페이지 구현하기 #96

Merged
merged 29 commits into from
Jul 18, 2022

Conversation

alstjr0183
Copy link

@alstjr0183 alstjr0183 commented Jul 12, 2022

안녕하세요 선생님 이번한주도 잘부탁드리겠습니다!
현재는 기본적인 틀구현까지만 진행해보았는대요 테스트 케이스추가할부분이많이 남아있습니다
현재 진행된곳까지 리뷰 부탁드릴게요!

react-router-dom 라이브러리를 설치했습니다
헤더쪽에 Link를 사용하기위해 최상단에 BrowserRouter를 설정해주었습니다
헤더를 구현하기위해 테스트코드를작성했습니다
헤더를 구현했습니다
6버전으로 깔려서 5버전으로 수정했습니다
Homepage의 렌더링확인  테스트 코드를 작성하였습니다
테스트코드를 토대로 Homepage를 구현하였습니다
Link 옵션을 href -> to 로 변경하였습니다
About의 렌더링 테스트코드를 작성하였습니다
About 컴포넌트를 구현하였습니다
Homepage -> About으로수정했습니다
Routes의 test를 작성했습니다
페이지를 연결하는 Routes를 구현하였습니다
about 쪽 테스트 설명에 오타가있어 수정했습니다
App에 Routes를 적용해주었습니다
Routes.test의 렌더링되는케이스 코드를 추가작성했습니다
About 파일의 이름을 변경해주었습니다
헤더쪽에 header태그로 감싸주었습니다
app의 수정으로인한 test 코드도 수정하였습니다
restaurants 카테고리의 테스트케이스를 작성하였습니다
테스트코드를 토대로 구현하였습니다
restaurants url에선 RestaurantsPage가 나오게끔 라우트 연결을 해주었습니다
Copy link

@daadaadaah daadaadaah left a comment

Choose a reason for hiding this comment

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

일단 진행된 것까지 PR 만드신거 너무 잘하셨어요!👍
곳곳에 About이 아닌 AboutPage 라고 되어있어야 하는 부분이 있는데, 이부분 수정해주시고~ 계속 진행해주세요🚀

Comment on lines +10 to +11
const SEOUL = REGIONS[0];
const KOREAN_FOOD = CATEGORIES[0];

Choose a reason for hiding this comment

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

오! 테스트 데이터를 변수명으로 의도를 드러내니까 테스트에 대해 좀더 명확히 생각할 수 있어서 좋은 것 같아요!👍 저도 나중에 활용해봐야겠어요😊

Copy link
Author

Choose a reason for hiding this comment

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

앗 그부분은 기본과제에 셋팅되어있는거 토대로 사용한거여서 제가 생각한방법은 아닙니다..!! 하하..

src/Routes.jsx Outdated
import Homepage from './Homepage';
import RestaurantsPage from './RestaurantsPage';

export default function Routes() {

Choose a reason for hiding this comment

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

Routes로 따로 빼서 관리하면 어떤 장점이 있나요?ㅎㅎ
저는 Routes로 빼는 것보다 App에서 관리하는게 이 앱에 어떤 페이지가 있는지 한눈에 알 수 있어서 App에서 관리하는 편이거든요!

Copy link
Author

Choose a reason for hiding this comment

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

저는 헤더쪽이랑 라우팅되는페이지쪽이랑 분리하고싶어서 그랬었는데 app에서 관리하게끔 다시 수정했습니다!!

));
}

it('RegionsContainer가 렌더링된다', () => {

Choose a reason for hiding this comment

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

좀더 의도를 드러내 보는 건 어떠세요?
예를 들어, 지역 이름이 보인다 또는 지역 버튼이 렌더링 된다 등으로!

Copy link
Author

Choose a reason for hiding this comment

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

수정했습니다

지역버튼이 렌더링된다 ~~~ 이런식으로 추가해주었습니다!!

라우팅하는쪽은 분리했었는데 다시 app쪽으로 합쳤습니다
테스트코드 케이스를 수정했습니다
@alstjr0183
Copy link
Author

안녕하세요 오늘은 피드백 주신거밖에 아직 수정을 못했습니다..ㅠ

질문이있습니다

api 호출시 cors 에러가 뜨는데 api쪽 문제일까요?!
image

페이지가없을때의 테스트코드를 작성했습니다
페이지가없을때를 구현했습니다
restauransPage의 테스트 케이스와 페이지가없을때 테스트 케이스를 추가했습니다
페이지가없을때의 페이지를 구현했습니다
@alstjr0183
Copy link
Author

안녕하세요 오늘은 페이지가없을때랑 app의 테스트코드를 보완했습니다!

@daadaadaah
Copy link

daadaadaah commented Jul 14, 2022

잘하셨습니다!ㅎㅎ라우터 버전 6으로도 한번 도전해보세요!🚀
디스코드 채널에 올려드린 자료 한번 보시고 적용시켜보시구요~!😄

@alstjr0183
Copy link
Author

alstjr0183 commented Jul 17, 2022

안녕하세요 선생님!
여행을 다녀오느라 진행을 못했네요 ㅠㅠ 이번주도 감사했습니다..! 강의보고 복습하겠습니다

@daadaadaah
Copy link

좋습니당!! 과제 완성 못하셨지만 회고 하신 점 너무 잘하셨어요👍
강의 꼭 들으시고~ 다음주에 다시 달려봅시당🚀

@daadaadaah daadaadaah merged commit 023500e into CodeSoom:alstjr0183 Jul 18, 2022
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