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

#134 헤더값 변경 #135

Merged
merged 5 commits into from
Feb 17, 2024
Merged

#134 헤더값 변경 #135

merged 5 commits into from
Feb 17, 2024

Conversation

Byun-Sung-Ho
Copy link
Contributor

📍 PR 타입 (하나 이상 선택)

  • 기능 추가
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타 사소한 수정

❗️ 관련 이슈 링크

#134

📌 개요

  • 마이프로필 대표 설정/취소 헤더변경
  • 프로필보관함 조회/ 특정 마이프로필 삭제 헤더 변경

🔁 변경 사항

  • 마이프로필 대표 설정/취소 헤더변경
  • 프로필보관함 조회/ 특정 마이프로필 삭제 헤더 변경
    -> 멤버아이디에서 토큰으로 변경

📸 스크린샷

👀 기타 더 이야기해볼 점

✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@Byun-Sung-Ho Byun-Sung-Ho linked an issue Feb 16, 2024 that may be closed by this pull request
1 task
@Byun-Sung-Ho Byun-Sung-Ho self-assigned this Feb 16, 2024
@Byun-Sung-Ho Byun-Sung-Ho added 🚧 update 코드 수정 ♻️ refactor 프로덕션 코드 리팩토링 labels Feb 16, 2024
Copy link
Contributor

@choign3123 choign3123 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@Byun-Sung-Ho Byun-Sung-Ho merged commit 53f3dec into main Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 프로덕션 코드 리팩토링 🚧 update 코드 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

#134 header의 member-id를 token으로 변경
2 participants