Skip to content

[FEAT] 워크스페이스 프로필 및 나의 프로필 API 세팅#111

Merged
jinj00oo merged 6 commits into
developfrom
feature/#102-setting-profile-api
Aug 1, 2025
Merged

[FEAT] 워크스페이스 프로필 및 나의 프로필 API 세팅#111
jinj00oo merged 6 commits into
developfrom
feature/#102-setting-profile-api

Conversation

@jinj00oo

@jinj00oo jinj00oo commented Aug 1, 2025

Copy link
Copy Markdown
Collaborator

체크리스트

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

📌 관련 이슈번호


✅ Key Changes

  • 워크스페이스 프로필 API 세팅
    • 워크스페이스 프로필 조회
  • 마이 프로필 API 세팅
    • 마이 프로필 조회
    • 마이 프로필 이미지 수정 및 삭제

💬 To Reviewers

  • 현재 응답 필드 또는 내부 변수에서 name / teamName, imageUrl / profileImage 등 동일한 개념을 가리키는 용어가 혼용되어 있어 백엔드에 통일 요청 드릴 예정입니다.
  • 로그아웃 및 회원탈퇴는 명세서가 완성되지 않아, 추후 구현 예정입니다.

- Sidebar 전용 Team 타입 정의를 위해 type Team 추출
- 팀 순서 변경 API 호출 위치를 주석으로 명시
- 호출 로직은 추후 mutation 연동 예정
@github-actions

github-actions Bot commented Aug 1, 2025

Copy link
Copy Markdown

@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.

확인했습니다! 아래 수정사항만 반영해주시면 될 것 같습니다~

import queryClient from '../../utils/queryClient';
import { queryKey } from '../../constants/queryKey';

// 사이드바, 설정 - 팀 순서 변경

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.

p2 : 진주님 apis/setting 폴더 속 함수들 위에 이 부분처럼 주석 달아주셔야 합니다! 다른 파일들도 수정해서 푸쉬해주세요!

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

추가하여 푸쉬하였습니다. 알려주셔서 감사합니다!

@HiJuwon HiJuwon 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.

확인했습니다, api 함수들 컨벤션도 잘 지켜주시고 깔끔하네요 bb 수고많으셨어요!

@github-actions

github-actions Bot commented Aug 1, 2025

Copy link
Copy Markdown

@jinj00oo
jinj00oo merged commit a3fee24 into develop Aug 1, 2025
1 check passed
@jinj00oo
jinj00oo deleted the feature/#102-setting-profile-api branch August 1, 2025 13:28
@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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants