Skip to content

[FEAT] 팀 목록 순서 변경 API 세팅#110

Merged
jinj00oo merged 3 commits into
developfrom
feature/#101-sidebar-api
Aug 1, 2025
Merged

[FEAT] 팀 목록 순서 변경 API 세팅#110
jinj00oo merged 3 commits into
developfrom
feature/#101-sidebar-api

Conversation

@jinj00oo

@jinj00oo jinj00oo commented Aug 1, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

체크리스트

  • 🧰 npm run dev로 실행 환경에서 잘 돌아가는걸 확인했나요?
  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙆 리뷰어는 등록했나요?

📌 관련 이슈번호

✅ Key Changes

  • 팀 목록 순서 변경 API 호출 함수 작성

    • 경로: src/apis/setting/usePatchWorkspaceTeams.ts
    • useMutation을 활용한 데이터 수정 요청 처리

  • 팀 순서 변경 API 호출 위치 주석으로 명시 (후속 연동 대비)

  • 응답 데이터 타입 리팩토링

    • 사이드바 내부에서 사용하는 타입을 위해 src/components/Sidebar/types.ts 파일 생성
    • 기존 TeamListResponse에서 필요한 속성만 Pick하여 재사용성 향상

💬 To Reviewers

  • 해당 API가 내부적으로 인증된 사용자 정보를 기반으로 동작하고 있어, 임시로 시큐리티 설정이 열려 있더라도 JWT 없이 호출 시 NullPointerException(NPE)가 발생한다고 백엔드에서 안내 받았습니다. 구글 로그인 로직이 마무리되어 실제 토큰을 포함한 요청이 가능해진 이후 연동 예정입니다. 일단 호출 위치는 주석으로 명시해두었습니다.
  • 또한, API 응답에서 profileUrl 등 일부 필드의 nullable 여부에 대한 명시가 필요해 보여 해당 내용을 백엔드 측에 전달드렸습니다.

jinj00oo added 3 commits August 1, 2025 17:16
@jinj00oo
#101 [FEAT] 팀 순서 변경 API 세팅
d724d53
@jinj00oo
Merge branch 'develop' of https://github.com/vecosystem/VECO_Frontend …
738a91d 
…into feature/#101-sidebar-api
@jinj00oo
#101 [REFACTOR] 팀 타입 추출 및 API 호출 위치 주석 처리
3a41f7b 
- Sidebar 전용 Team 타입 정의를 위해 type Team 추출
- 팀 순서 변경 API 호출 위치를 주석으로 명시
- 호출 로직은 추후 mutation 연동 예정
@jinj00oo jinj00oo self-assigned this Aug 1, 2025
@jinj00oo jinj00oo added ✨ FEAT 기능 구현 💎 진주 labels Aug 1, 2025
@github-actions

github-actions Bot commented Aug 1, 2025

Copy link
Copy Markdown

✅ PREVIEW https://veco-nubidogbx-gaeulzzangs-projects.vercel.app

gaeulzzang
gaeulzzang approved these changes Aug 1, 2025
View reviewed changes

@gaeulzzang gaeulzzang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

주석 부분 확인했습니다~!!

waldls
waldls approved these changes Aug 1, 2025
View reviewed changes

@waldls waldls left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

주석 확인했습니다! 문서 오류 해결 되면 연동 진행하시면 될 것 같습니다! 수고하셨어요!

@jinj00oo
jinj00oo merged commit b6cd8c0 into develop Aug 1, 2025
1 check passed
@jinj00oo
jinj00oo deleted the feature/#101-sidebar-api branch August 1, 2025 13:27
@HiJuwon HiJuwon added this to the Demo Day milestone Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gaeulzzang gaeulzzang gaeulzzang approved these changes
@waldls waldls waldls approved these changes
@sunhwaaRj sunhwaaRj Awaiting requested review from sunhwaaRj sunhwaaRj is a code owner
@HiJuwon HiJuwon Awaiting requested review from HiJuwon HiJuwon is a code owner

Assignees

@jinj00oo jinj00oo

Labels

✨ FEAT 기능 구현 💎 진주

Projects

None yet

Milestone

Demo Day

Development

Successfully merging this pull request may close these issues.

[FEAT] 팀 목록 순서 수정 API 세팅

4 participants

@jinj00oo @gaeulzzang @waldls @HiJuwon