[Fix] 나의 과제 페이지, 수강 신청 페이지 QA 반영 #133

요거 살펴봤을 때 서버 컴포넌트에서 캐싱된 데이터를 사용할 때 불러오지 못하는 발생하는 문제인 거 같아요... 관련이슈 에서도 로컬에서는 괜찮은데 prod 환경에서 발생한다고 하더라구요.
우선 해결방법으로는 서버 컴포넌트에서는 캐싱하지 않거나 (cache: no-store) 기존의 서버 컴포넌트를 클라이언트 컴포넌트로 다 바꾸는 건데 force-cache 로 가능한 것들을 no-store 로 사용해야 하느 부분이 좀 찝찝하긴 한데 간헐적으로 발생하는 문제를 우선 해결해보려면 no-cache 로 가야될 거 같긴 함니다..

+) 추가 QA

나의 과제 페이지에서 네트워크 요청이 너무 많이 들어가는 문제가 있었습니다. (onGoing, curriculum) 과제 제출 박스에서 disabled 를 처리하기 위함이었는데요, 과제 박스가 많아질 수록 요청 수가 더욱 많아질 거 같다는 생각이 들어 상위 부모 컴포넌트 AssignmentBox 서버 컴포넌트에서 해당 api 들을 부르고 props 로 전달하는 방식으로 리팩토링 합니다.

ghdtjgus76

아래 부분 아직 반영 안 된 거 같아서 확인 부탁드려요!
GitHub 레포지터리

그리고 이런 이슈가 있는데 이 부분도 수정이 필요할 거 같습니다!

apps/client/app/(afterLogin)/study-apply/_components/StudyItem.tsx

ghdtjgus76 · 2024-09-06T18:13:17Z

일단 언급하셨던 나의 과제 페이지 관련 이슈는 no-store로 해놓고 추후 더 나은 방법을 찾아봐야 할 거 같습니다...

SeieunYoo · 2024-09-07T05:55:58Z

앗 원래 제출 실패가 뜨면 제출 실패 사유랑 같이 팝업이 떠야 하는데 과제 제출이 요상하게 된 거 같네요...?! 혹시 public 레포에 있는 건가요?

+) 문구는 수정했습니다 a1465de @ghdtjgus76

ghdtjgus76 · 2024-09-07T06:02:51Z

앗 원래 제출 실패가 뜨면 제출 실패 사유랑 같이 팝업이 떠야 하는데 과제 제출이 요상하게 된 거 같네요...?! 혹시 public 레포에 있는 건가요?

+) 문구는 수정했습니다 a1465de @ghdtjgus76

음 저때 제출하고 레포를 삭제하긴 했었는데 그 문제 때문인지 모르겠네요..

SeieunYoo · 2024-09-07T06:10:39Z

아하,, 과제 제출하고 레포 삭제했을 때의 테스트는 해보지 않았네용 저도 확인해볼게요!
+) 관련해서 문의 남겼습니다.

apps/client/utils/getAssignmentGithubFolderName.ts

apps/client/app/(afterLogin)/my-study/my-assignment/_components/AssignmentContent/index.tsx

ghdtjgus76

수고하셨습니다!
근데 레포지토리 입력란 에러 텍스트 부분들이 해요체로 변경돼야 할 거 같습니다..!

eugene028

이전에 나왔던 QA 사항은 거의 다 반영하셨군용 수고하셨습니다 ! 👏
슬랙에서 논의 중인 사항만 마무리되면 확인 이후 머지하시면 될 것 같아용!

* fix: 스터디 신청 기간 날짜 겹치지 않도록 수정 (#134) * [Fix] 스터디 목록 조회를 멘토용 조회 API로 수정해요. (#136) * fix: 스터디 목록 조회 멘토용으로 수정 * fix: 사용하지 않는 DTO 제거 * chore: unuse 변수 제거 * fix: 코드리뷰 반영 * [Chore] main 브랜치와 dev 브랜치 commit head 정렬 (#138) * fix: 스터디 신청 기간 날짜 겹치지 않도록 수정 (#134) (#135) * [Deploy] 멘토용 스터디 목록 조회 (2.0.4) (#137) * fix: 스터디 신청 기간 날짜 겹치지 않도록 수정 (#134) * [Fix] 스터디 목록 조회를 멘토용 조회 API로 수정해요. (#136) * fix: 스터디 목록 조회 멘토용으로 수정 * fix: 사용하지 않는 DTO 제거 * chore: unuse 변수 제거 * fix: 코드리뷰 반영 * [Fix] 나의 과제 페이지, 수강 신청 페이지 QA 반영 (#133) * feat: 큐에이 반영 * fix: 소문자로 변경 * refactor: api dto 반영, 이번 주 과제 조회 api 연결 * feat: 히스토리 api 수정된 dto 반영 * feat: 히스토리 api 수정된 거 반영, 레포지토리 입력 api 반영 * refactor: 레포 모달 창 상태 관리 컴포넌트로 벼녕 * feat: 필요없는 로직 삭제 * feat: 이번 주 과제 조회 스터디 추가 * feat: 모달 라우팅 삭제 * feat: submissionLink 에서 폴더 * chore: 안쓰는 상수 삭제 * chore: 말줄임 처리, github url 인지 체크 * feat: 스터디 시작 날짜가 현재 날짜인지 확인하고 disabled 처리 * feat: 공통 타입은 common 으로 옮기기, 리뷰 반영 * fix: 빠진 네이밍 수정 반영 * feat: Button asProp 으로 Link 전달, css 수정 * refactor: NonNullable 처리, Initial 로 관리 * feat: github 관련 유틸 분리,css 수정 * chore: 필요없는 프래그먼트 삭제 * chore: cache 속서 추가 * feat: currentPath href 매치될 때만 active 되게 navItem 수정 * chore: myStudy 용 타입 추가 * chore: EDITING_WITH_WARNING 타입 삭제, 상태 네이밍 변경 * chore: 조건식 간소화 * chore: submissionLink null 로 내려오는 부분 우선 삭제 * feat: 리뷰 반영 * feat: 수강 신청 아이템 width 고정 * fix: dev 환경일 때는 dev-onboarding 으로 연결 * feat: 부모컴포넌트에서 호출하도록 수정 * feat: 히스토리 아이템 반응형 처리 * feat: upcoming api 로직 삭제 * feat: 부모컴포넌트에서 api 불러오기 * fix: 컴포넌트 호출 위치 수정 * chore: 문구 수정 반영 * chore: 헤더 삭제 * chore: 수강신청 아이템 정렬 수정 * chore: repo link 가 있는 지 확인하는 로직 추가 * chore: 레포지토리로 네이밍 변경 * chore: 인자명 repositoryLink 로 변경 * chore: length 0일때만 확인 * fix: cache no-store 로 설정 * chore: api 호출하지 않고 props 로 전달하는 걸로 수정 * chore: 목데이터 삭제 * chore: 에러 메시지 문구 해요체로 변경 * chore: 컬러 변경, 함수 네이밍 변경 * feat: unknown 인 케이스 우선 임의로 반영 * chore: 실패 팝오버 문구 수정 * chore: popoverRef 위치 상단으로 변경 * [Fix] 수강 신청 페이지 반응형 수정, 과제 없을 때 UI 수정 (#141) * feat: 큐에이 반영 * fix: 소문자로 변경 * refactor: api dto 반영, 이번 주 과제 조회 api 연결 * feat: 히스토리 api 수정된 dto 반영 * feat: 히스토리 api 수정된 거 반영, 레포지토리 입력 api 반영 * refactor: 레포 모달 창 상태 관리 컴포넌트로 벼녕 * feat: 필요없는 로직 삭제 * feat: 이번 주 과제 조회 스터디 추가 * feat: 모달 라우팅 삭제 * feat: submissionLink 에서 폴더 * chore: 안쓰는 상수 삭제 * chore: 말줄임 처리, github url 인지 체크 * feat: 스터디 시작 날짜가 현재 날짜인지 확인하고 disabled 처리 * feat: 공통 타입은 common 으로 옮기기, 리뷰 반영 * fix: 빠진 네이밍 수정 반영 * feat: Button asProp 으로 Link 전달, css 수정 * refactor: NonNullable 처리, Initial 로 관리 * feat: github 관련 유틸 분리,css 수정 * chore: 필요없는 프래그먼트 삭제 * chore: cache 속서 추가 * feat: currentPath href 매치될 때만 active 되게 navItem 수정 * chore: myStudy 용 타입 추가 * chore: EDITING_WITH_WARNING 타입 삭제, 상태 네이밍 변경 * chore: 조건식 간소화 * chore: submissionLink null 로 내려오는 부분 우선 삭제 * feat: 리뷰 반영 * feat: 수강 신청 아이템 width 고정 * fix: dev 환경일 때는 dev-onboarding 으로 연결 * feat: 부모컴포넌트에서 호출하도록 수정 * feat: 히스토리 아이템 반응형 처리 * feat: upcoming api 로직 삭제 * feat: 부모컴포넌트에서 api 불러오기 * fix: 컴포넌트 호출 위치 수정 * chore: 문구 수정 반영 * chore: 헤더 삭제 * chore: 수강신청 아이템 정렬 수정 * chore: repo link 가 있는 지 확인하는 로직 추가 * chore: 레포지토리로 네이밍 변경 * chore: 인자명 repositoryLink 로 변경 * chore: length 0일때만 확인 * fix: cache no-store 로 설정 * chore: api 호출하지 않고 props 로 전달하는 걸로 수정 * chore: 목데이터 삭제 * chore: 에러 메시지 문구 해요체로 변경 * chore: 컬러 변경, 함수 네이밍 변경 * feat: unknown 인 케이스 우선 임의로 반영 * chore: 실패 팝오버 문구 수정 * chore: popoverRef 위치 상단으로 변경 * chore: 히스토리 아이템 반응형 수정 * feat: 과제 없을 때 ui 수정 --------- Co-authored-by: Eugene Kim <[email protected]>

SeieunYoo added 30 commits September 2, 2024 01:14

feat: 큐에이 반영

473c397

fix: 소문자로 변경

4d96c98

refactor: api dto 반영, 이번 주 과제 조회 api 연결

0a470e4

feat: 히스토리 api 수정된 dto 반영

15efae9

feat: 히스토리 api 수정된 거 반영, 레포지토리 입력 api 반영

98ca46f

refactor: 레포 모달 창 상태 관리 컴포넌트로 벼녕

14e3318

feat: 필요없는 로직 삭제

4a2bdaa

feat: 이번 주 과제 조회 스터디 추가

e4e81a7

feat: merge dev

02ccc06

feat: 모달 라우팅 삭제

c40f35d

feat: submissionLink 에서 폴더

3f55541

chore: 안쓰는 상수 삭제

c535042

chore: 말줄임 처리, github url 인지 체크

3ba2820

feat: 스터디 시작 날짜가 현재 날짜인지 확인하고 disabled 처리

30b1482

feat: 공통 타입은 common 으로 옮기기, 리뷰 반영

5300dab

fix: 빠진 네이밍 수정 반영

2353d51

feat: Button asProp 으로 Link 전달, css 수정

44b6eae

refactor: NonNullable 처리, Initial 로 관리

87913e4

feat: github 관련 유틸 분리,css 수정

9e1ec27

chore: 필요없는 프래그먼트 삭제

083fd28

chore: cache 속서 추가

9193df3

feat: currentPath href 매치될 때만 active 되게 navItem 수정

9807666

chore: myStudy 용 타입 추가

70a1a10

chore: EDITING_WITH_WARNING 타입 삭제, 상태 네이밍 변경

2f2237c

chore: 조건식 간소화

172f10c

chore: submissionLink null 로 내려오는 부분 우선 삭제

329f205

feat: 리뷰 반영

2d7b185

feat: 수강 신청 아이템 width 고정

690e48d

fix: dev 환경일 때는 dev-onboarding 으로 연결

1bacf2d

feat: 부모컴포넌트에서 호출하도록 수정

904a648

SeieunYoo requested review from eugene028 and hamo-o as code owners September 5, 2024 18:09

SeieunYoo added the 👩‍💻 리뷰 남겨주세요! label Sep 5, 2024

ghdtjgus76 reviewed Sep 6, 2024

View reviewed changes

apps/client/app/(afterLogin)/study-apply/_components/StudyItem.tsx Outdated Show resolved Hide resolved

chore: 수강신청 아이템 정렬 수정

005b1e1

eugene028 reviewed Sep 7, 2024

View reviewed changes

apps/client/utils/getAssignmentGithubFolderName.ts Outdated Show resolved Hide resolved

eugene028 reviewed Sep 7, 2024

View reviewed changes

apps/client/app/(afterLogin)/my-study/my-assignment/_components/AssignmentContent/index.tsx Outdated Show resolved Hide resolved

SeieunYoo added 7 commits September 7, 2024 15:44

chore: repo link 가 있는 지 확인하는 로직 추가

1cd5a71

chore: 레포지토리로 네이밍 변경

3bf13d9

chore: 인자명 repositoryLink 로 변경

40479c2

chore: length 0일때만 확인

76da218

fix: cache no-store 로 설정

11de24d

chore: api 호출하지 않고 props 로 전달하는 걸로 수정

ac4fcf5

chore: 목데이터 삭제

07d4ca7

SeieunYoo requested review from eugene028 and ghdtjgus76 September 7, 2024 07:10

ghdtjgus76 approved these changes Sep 7, 2024

View reviewed changes

eugene028 approved these changes Sep 7, 2024

View reviewed changes

SeieunYoo added 5 commits September 7, 2024 16:21

chore: 에러 메시지 문구 해요체로 변경

863b435

chore: 컬러 변경, 함수 네이밍 변경

963e8ff

feat: unknown 인 케이스 우선 임의로 반영

017758a

chore: 실패 팝오버 문구 수정

2f614a6

chore: popoverRef 위치 상단으로 변경

b4d1a12

SeieunYoo merged commit 0e8bbb3 into dev Sep 7, 2024
1 check passed

SeieunYoo deleted the fix/seieun-0904-qa branch September 7, 2024 12:43

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Fix] 나의 과제 페이지, 수강 신청 페이지 QA 반영 #133

[Fix] 나의 과제 페이지, 수강 신청 페이지 QA 반영 #133

SeieunYoo commented Sep 4, 2024 •

edited

Loading

ghdtjgus76 left a comment

ghdtjgus76 commented Sep 6, 2024

SeieunYoo commented Sep 7, 2024 •

edited

Loading

ghdtjgus76 commented Sep 7, 2024

SeieunYoo commented Sep 7, 2024 •

edited

Loading

ghdtjgus76 left a comment

eugene028 left a comment

[Fix] 나의 과제 페이지, 수강 신청 페이지 QA 반영 #133

[Fix] 나의 과제 페이지, 수강 신청 페이지 QA 반영 #133

Conversation

SeieunYoo commented Sep 4, 2024 • edited Loading

🎉 변경 사항

🚩 관련 이슈

🙏 여기는 꼭 봐주세요

ghdtjgus76 left a comment

Choose a reason for hiding this comment

ghdtjgus76 commented Sep 6, 2024

SeieunYoo commented Sep 7, 2024 • edited Loading

ghdtjgus76 commented Sep 7, 2024

SeieunYoo commented Sep 7, 2024 • edited Loading

ghdtjgus76 left a comment

Choose a reason for hiding this comment

eugene028 left a comment

Choose a reason for hiding this comment

SeieunYoo commented Sep 4, 2024 •

edited

Loading

SeieunYoo commented Sep 7, 2024 •

edited

Loading

SeieunYoo commented Sep 7, 2024 •

edited

Loading