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

chacha-FE-lecture1,2,3 #1

Open
wants to merge 38 commits into
base: main
Choose a base branch
from
Open

chacha-FE-lecture1,2,3 #1

wants to merge 38 commits into from

Conversation

ChaeyeonAhn
Copy link
Collaborator

@ChaeyeonAhn ChaeyeonAhn commented Jul 23, 2024

요약

Lecture1 과제

컴포넌트 구현하고, page.tsx에 간단히 테스트해볼 수 있도록 만들어 두었습니다.

다른 것은 문제 없이 구현된 것 같은데 ItemNumberInput 컴포넌트에서
value={shownValue} 로 지정하면 입력창에 무언가를 입력한 뒤 backspace가 안 먹히고,
value={input} 로 지정하면 backspace는 먹히는데 '개' 가 안 붙는다는 문제를 해결하지 못했습니다

Lecture2 과제

컴포넌트들을 가져다가 ChachaMyInfoFrame, ChachaMyClubFrame, ChachaMyServiceFrame을 구현했습니다.
Toggle, 그리고 전체보기 혹은 내역 더보기(TextButton)를 눌렀을 시에 다른 페이지로 라우팅 되는 것도 구현했습니다.

Lecture3 과제

먼저, mockup 데이터를 만들었습니다.
chachaMockMyClub.ts, chachaMockUpPhone.ts 두 개를 만들었습니다.
이것을 만드는 과정에서는 api endpoint 경우들 중 우리의 프레임에서 필요한 정보를 가져오는 endpoint를 찾아서 그것을 import한 뒤,
그 파일에 들어있는 zod 타입 형식에 맞춰서 mock 데이터를 만들었습니다.
그 뒤에는 frame으로 mock data를 가져오는 식으로 진행했습니다.

특히 phone number 목 데이터를 가져올 때에는 import 형식이 파일 > mock data 변수 > 안의 요소 (phoneNumber) 이런 식으로 가져와야 오류가 안 생기는 것에 대해서 의문이 생겼습니다. 또, clubs에서 이미 만들어 놓으신 mockMyClub.ts에 offset이라는 것이 있던데 이것은 무슨 역할을 하는지 의문이 생겼습니다.

8/10 Update

리뷰 달아주신 내용들 수정했습니다!

  • 특히 /chacha/my/clubs 페이지에서 FoldableSectionTitle 안 접히는 문제에 대해서는 useState로 토글을 정의하면 모든 학기의 접기/펼치기가 동기화 되는 문제가 있었는데, ClubsSectionTitle을 써서 각 학기에 대한 토글을 따로 제공하니까 문제가 해결되어 이 점을 배울 수 있었습니다.
  • service 코드 구현 완료했습니다! 다만 phoneNumber 가져오는 건 되는데 수정이 안 됩니다.. event.target이 undefined로 뜨는 문제가 있습니다.

@ChaeyeonAhn ChaeyeonAhn self-assigned this Jul 23, 2024
@ChaeyeonAhn ChaeyeonAhn changed the title FE-Seminar Assignment#1 chacha FE-Seminar Assignment-1 chacha Jul 23, 2024
@ChaeyeonAhn ChaeyeonAhn changed the title FE-Seminar Assignment-1 chacha chacha-FE-lecture1 Jul 23, 2024
@ChaeyeonAhn ChaeyeonAhn changed the title chacha-FE-lecture1 chacha-FE-lecture1,2 Jul 27, 2024
Copy link
Collaborator

@wjeongchoi wjeongchoi left a comment

Choose a reason for hiding this comment

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

1차시 과제 리뷰 먼저 했습니다!

단위랑 커서 문제는 기존 코드 보면서 어떻게 하면 될지 공부해봐도 좋을 것 같아요

코멘트도 확인해주세요 수고했습니다!

@ChaeyeonAhn ChaeyeonAhn changed the title chacha-FE-lecture1,2 chacha-FE-lecture1,2,3 Aug 2, 2024
Copy link
Collaborator

@wjeongchoi wjeongchoi left a comment

Choose a reason for hiding this comment

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

과제하느라 수고 많았습니다!!
service 코드 이용해서 값 가져오는거는 구현이 안 되어 있는 것으로 보이는데, 혹시 마저 마무리해보고 싶다면 작업한 후 저에게 알려주세요!

return useQuery<ISuccessResponseType, Error>({
queryKey: [apiPrt005.url(), requestQuery], // Query Param 도 같이 줘야 함
queryFn: async (): Promise<ISuccessResponseType> => {
try {
Copy link
Collaborator

Choose a reason for hiding this comment

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

try catch 안 써도 됩니당

return useQuery<ISuccessResponseType, Error>({
queryKey: [apiUsr001.url(), requestQuery],
queryFn: async (): Promise<ISuccessResponseType> => {
try {
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도! try catch 안 써도 status로 에러 생긴거 걸러서 괜찮아요

return useQuery<ISuccessResponseType, Error>({
queryKey: [apiAcf007.url(), requestQuery], // Query Param 도 같이 줘야 함
queryFn: async (): Promise<ISuccessResponseType> => {
try {
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기도 try catch 쓰지 않고 해보기!

Comment on lines +56 to +57
toggle={toggle}
toggleHandler={() => setToggle(!toggle)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 toggle 안 써도 작동된대요!

Comment on lines +46 to +49
toggle={toggle}
toggleHandler={() => setToggle(!toggle)}
/>
{toggle && (
Copy link
Collaborator

Choose a reason for hiding this comment

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

요기도 toggle 없이 foldableSectionTitle의 children으로 넣어주면 됩니다

Comment on lines +30 to +32
const MyClubTitleWrapper = styled.div`
padding-left: 24px;
`;
Copy link
Collaborator

Choose a reason for hiding this comment

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

이건 foldableSectionTitle 안에 들어가면 없어도 왼쪽 간격 적용될거에요

<PhoneInput
label="전화번호"
value={phone}
onChange={handlePhoneChange}
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기에 바로 setPhone 넣으면 handlePhoneChange 없이 할 수 있어요!

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