Skip to content

Conversation

@mirlee0304
Copy link
Member

@mirlee0304 mirlee0304 commented Feb 20, 2025

Pull request

Related issue

#248

Motivation and context

userId를 passport를 이용해 가져오도록 설정 파일들을 수정하였습니다.
프로필 수정 api 요청 포맷을 수정하였습니다.

Solution

How has this been tested

스크린샷 2025-02-20 오후 1 42 33

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the docs/CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@netlify
Copy link

netlify bot commented Feb 20, 2025

Deploy Preview for jootalkpia ready!

Name Link
🔨 Latest commit 4dd84e6
🔍 Latest deploy log https://app.netlify.com/sites/jootalkpia/deploys/67b6b2dfce7941000878bcf9
😎 Deploy Preview https://deploy-preview-252--jootalkpia.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mirlee0304 mirlee0304 self-assigned this Feb 20, 2025
@mirlee0304 mirlee0304 added ✨ Feature 기능 추가 🎮 BE 백엔드 무조건 스프린트내에 해야하는 것들 labels Feb 20, 2025
@mirlee0304 mirlee0304 added this to the 주톡피아 마일스톤3 milestone Feb 20, 2025
Copy link
Collaborator

@ki-met-hoon ki-met-hoon left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mirlee0304 mirlee0304 merged commit e7d23d6 into dev Feb 20, 2025
15 checks passed
@mirlee0304 mirlee0304 deleted the be-feat/file-profile-passport branch February 25, 2025 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 추가 무조건 스프린트내에 해야하는 것들 🎮 BE 백엔드

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants