Skip to content

Conversation

@Minkyu0424
Copy link
Contributor

#️⃣연관된 티켓 넘버

CLAP-191
close #54

📝작업 내용

연결한 API는 다음과 같습니다.
요청 생성, 1차 카테고리 조회, 2차 카테고리 조회

추가로 작업을 하며 카테고리 드롭다운을 공통 드롭다운으로 분리했습니다.
axios 인스턴스를 두 개로 쪼갰습니다. 하나는 json, 하나는 formdata를 보내는 형식으로

이 과정에서 axios 인스턴스를 불러오는 코드가
import axiosInstance from '../utils/axios'
에서
import { axiosInstance } from '../utils/axios'
로 변경되었고 @seorang42 님이 작성한 코드에는 반영해두었습니다.

🏞️스크린샷 (선택)

작업 내용에 대한 스크린샷을 포함해주세요

💬리뷰 요구사항(선택)

기존의 드롭다운은 유지하고 카테고리 드롭다운을 만든거라 RequestTaskDropdown을 사용중이시라면 코드 확인 후 사용해주시면 됩니다.
placeholder를 제거하고 라벨의 이름 + 입력 or 선택해주세요 같은 구성으로 리팩토링 했습니다
라벨 타입이 바뀐건에 대해선 다음 PR에서 반영해 머지하도록 하겠습니다!
axios 인스턴스를 만들어 api 호출 코드를 api 폴더에 역할별로 나누어 api 함수를 호출해 값을 받는 방식을 사용했는데 @seorang42 님은 코드내에 쓰셔서이 방식도 통일하면 좋을 것 같습니다

@Minkyu0424 Minkyu0424 added ✨ Feature 기능 개발 📬 API 서버 API 통신 labels Feb 2, 2025
@Minkyu0424 Minkyu0424 self-assigned this Feb 2, 2025
Copy link
Contributor

@seorang42 seorang42 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

<RequestTaskDropdown
<CategoryDropDown
v-model="category1"
:options="DUMMY_REQUEST_TASK_CATEGORIES"
Copy link
Contributor

Choose a reason for hiding this comment

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

더미 데이터는 삭제하고 있으신 걸까요?
사용되지 않을 더미 데이터는 확인 후 삭제 부탁드립니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아직 1,2차 카테고리를 사용하는 페이지들에 전부 반영하지 않아서 (요청 승인) 해당 파트를 진행하며 마지막으로 지우려합니다!

@Minkyu0424 Minkyu0424 merged commit 019e0f4 into develop Feb 2, 2025
@Minkyu0424 Minkyu0424 deleted the CLAP-191 branch February 3, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📬 API 서버 API 통신 ✨ Feature 기능 개발

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-191 요청 생성 API 연결

3 participants