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

fix: api 변경에 따른 수정 작업 #162

Merged
merged 4 commits into from
Dec 4, 2024
Merged

fix: api 변경에 따른 수정 작업 #162

merged 4 commits into from
Dec 4, 2024

Conversation

gahyuun
Copy link
Member

@gahyuun gahyuun commented Dec 4, 2024

📌 작업 내용

closed #161

구현 내용 및 작업 했던 내역

  • api 변경에 따른 수정 작업
  • table nonrenderable key 추가

🤔 고민 했던 부분

  • 서버에서 과목 검색과 과목 상세에서 id가 정수형이 아닌 과목 코드로 변경이 되었고 이에 따른 타입 수정을 진행하였습니다
  • 또한 기존에는 id가 정수형이기에 Table 공통 컴포넌트에서 id면 렌더링하지 않도록 처리해주었습니다 하지만 수정 이후 마이페이지 기이수 과목 리스트에선 id가 정수형이어서 렌더링되지 않아야하지만, ResultCagegoryDetailLecture에서는 id가 과목 코드이므로 노출이 필요했습니다
  • 그래서 nonRenderableKey라는 속성을 통해 Table 컴포넌트를 사용하는 곳에서 자유롭게 처리하도록 수정했습니다

@gahyuun gahyuun self-assigned this Dec 4, 2024
Copy link
Collaborator

@seonghunYang seonghunYang left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Copy link
Member

@yougyung yougyung left a comment

Choose a reason for hiding this comment

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

급하게 진행되었는데, 빠르게 작업해주셔서 감사합니당. 수고하셨어요!

@gahyuun gahyuun merged commit 3d5ce15 into main Dec 4, 2024
1 of 3 checks passed
@gahyuun gahyuun deleted the fix-api/#161 branch December 4, 2024 11:37
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.

서버 api 변경에 따른 작업
3 participants