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

QA 수정사항 진행 #145

Merged
merged 12 commits into from
Nov 7, 2024
Merged

QA 수정사항 진행 #145

merged 12 commits into from
Nov 7, 2024

Conversation

Diwoni
Copy link
Collaborator

@Diwoni Diwoni commented Nov 5, 2024

#️⃣ 연관된 이슈

#144
#148
#149

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

마이페이지 QA 수정 반영

  • [QA] 마이페이지 수정 #144 참고
  • 특이사항 : 포인트 출금, 충전 요청 시 포인트 로그내역과 보유 포인트가 새로고침을 해야 떴었는데, invalidateQueries 를 사용해서 효율적으로 최신화된 데이터를 가져오도록 수정했습니다. 여러 곳에서 refetch를 호출할 필요 없이 한 곳에서 관리 가능하더라구요. 다른 곳(시니어 수정,삭제)에서는 컴포넌트에서 직접 refetch를 호출했는데, invalidateQueries를 사용하면 데이터 갱신 로직을 API hook 에서 한번에 처리할 수 있어 더 깔끔한 것 같습니다.

보호자 홈 화면 UI 수정

안부전화 수정 API 삭제

  • 백엔드분들의 요청에 따라 안부전화 수정 API 삭제했습니다.

스크린샷 (선택)

보호자 홈 화면 UI 수정

image

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

백엔드에서 페이지네이션으로 구현해서 클라이언트에서 페이지 번호 변경 시 해당 페이지의 데이터를 새로 가져오는 건 필수적인 것 같은데 그러면 새로고침? 같은 성능 문제에 대해 전체 데이터를 가져와서 프론트에서 페이징을 처리해야 하는 거보다 비효율적인 것 같다는 생각이 드는데 어떻게 생각하실까요?? 일단 의미 있을 지는 잘 모르겠지만 refetch 적용해놓았습니다. 최초에 데이터 가져온 뒤로는 페이지네이션 이동 시 새로고침 없이 빠르게 캐싱됩니다.

⏰ 현재 버그

✏ Git Close

close #144
close #148
close #149

@Diwoni Diwoni added the 😡 Bug Fix 발생한 버그를 수정하는 경우 label Nov 5, 2024
@Diwoni Diwoni requested review from Dobbymin and JYN523 November 5, 2024 02:08
@Diwoni Diwoni self-assigned this Nov 5, 2024
@Diwoni Diwoni linked an issue Nov 5, 2024 that may be closed by this pull request
12 tasks
@Diwoni Diwoni added the ✨ Feature 새로운 기능 추가 및 구현하는 경우 label Nov 5, 2024
@Diwoni Diwoni changed the title 계좌 등록 버튼 수정 QA 수정사항 진행 Nov 5, 2024
Copy link
Contributor

@Dobbymin Dobbymin left a comment

Choose a reason for hiding this comment

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

파일이 엄청 많네용... 고생하셨습니다.

) : isEditingAccount ? (
<Flex gap={2}>
<Button
w='100px'
h='40px'
fontSize='md'
colorScheme='teal'
fontSize='16px'
Copy link
Contributor

Choose a reason for hiding this comment

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

Chakra에서는 fontSize='md' 가 16px 인것 같더라구요

Comment on lines +13 to +15
onSuccess: () => {
authStorage.accessToken.set(undefined);
authStorage.refreshToken.set(undefined);
Copy link
Contributor

Choose a reason for hiding this comment

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

remove 하는게 낫지 않나요?? 저거 그 제가 잘 모른다고 했던 storage 라서 그런건가요?? undefined로 했을때 키가 남아있는지 확인해봐야할것같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

보니까 키도 같이 제거되더라구요! remove 도 좋지만 일단 Util 함수로 만들어둔 걸 사용하는 게 좋다고 생각해서 사용했습니다!

@Diwoni Diwoni merged commit be4f109 into Weekly Nov 7, 2024
1 check passed
@Diwoni Diwoni deleted the Fix/issue-#144 branch November 7, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😡 Bug Fix 발생한 버그를 수정하는 경우 ✨ Feature 새로운 기능 추가 및 구현하는 경우
Projects
None yet
2 participants