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

feat: 로드맵 페이지에 커리큘럼 컴포넌트 추가 및 API 연결 #1138

Merged
merged 13 commits into from
Dec 28, 2022

Conversation

wonsss
Copy link
Contributor

@wonsss wonsss commented Dec 18, 2022

로드맵-커리큘럼

MSW를 사용하여 API 명세대로 서버 모킹하여, mock data에서 상호작용 확인은 위와 같이 하였습니다.

그러나 로드맵 기능 관련한 백엔드 API에는 응답 데이터가 없어서 정상작동하는지 확인하지 못했습니다.

relate #1046

@wonsss wonsss requested a review from euijinkk December 18, 2022 22:15
@wonsss wonsss self-assigned this Dec 18, 2022
curriculumId -> name
테스트라는 목적에 맞게 mock data 단순화
@wonsss
Copy link
Contributor Author

wonsss commented Dec 18, 2022

@her0807 @euijinkk @wishoon
[현재 진행상황 테스트하여 확인]
아래와 같이, 로드맵 기능 관련한 백엔드 API에는 응답 데이터가 비어 있어서 정상작동하는지 확인하지 못했습니다.
Mock API와 연결하여 응답 데이터를 받으려면 어떻게 해야할까요?
그리고 커리큘럼별 Session 목록 조회는 아직 백엔드에서 구현되지 않은 것 같아 API 연결 시 에러가 발생하는 것 같습니다.

커리큘럼-백엔드 로컬 연결 결과

@her0807
Copy link
Contributor

her0807 commented Dec 19, 2022

@wonsss 마르코!!!! 믿고 있었다구... ㅎㅎ 우선 지금 커리큘럼 API 구현은 다 되어있는상태인데, 더미데이터가 없어서 마르코 말처럼 반환값이 따로 없는 상태라 더미데이터 추가해 드리겠습니다!!!

그리고 세션에서 스펙 변경된 부분은 백엔드 내부에서 리팩터링 작업 후 API 를 변경하려고 했으나, 현재 세션을 사용하고 있는 로직이 많아서 url 변경을 하면 프론트에서 변경해야할 코드가 많아질 것 같아서 파라미터로 안받고, body 에 데이터로 받기로 루키와 협의가 된 상태입니다.
그래서 url 변경은 따로 안해도 될 것 같은데 ,, 혹시 변경하셨나요?! 하셨으면 아직 세션 작업 시작안해서 변경하는 쪽으로 저희가 바꿔도 됩니다!!!

@sonarcloud
Copy link

sonarcloud bot commented Dec 23, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@wonsss
Copy link
Contributor Author

wonsss commented Dec 23, 2022

@her0807 @wishoon @euijinkk
수달과 루키가 논의해주신 대로 세션 API는 "기존 api는 두고, path variable도 추가된 api 추가하는" 방향이 좋을 것 같네요.

이후 루키가 #1139 PR에서 올린 추가된 세션 API를 확인해보니 프론트 개발부분이 연결이 되는 것 같습니다. 다만, 아직 응답 데이터가 비어있어서 클라이언트와 서버가 서로 잘 상호작용하는지는 추가적인 확인이 필요할 것 같네요.

image

말씀해주신 것처럼 mock data를 추가해주시면 전체적으로 잘 작동하는지까지 알 수 있을 것 같아요. 감사합니다.

Copy link
Contributor

@euijinkk euijinkk left a comment

Choose a reason for hiding this comment

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

기존 방식대로 잘 구현된 것 같습니다.

Query key 상세화한 점 좋았습니다!

고생하셨습니다.

@wonsss wonsss merged commit 31a80e2 into develop Dec 28, 2022
@seokhongkim seokhongkim deleted the feature/1046-curriculum branch January 20, 2023 07:57
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.

3 participants