Skip to content

Conversation

@l-lyun
Copy link
Contributor

@l-lyun l-lyun commented Jul 17, 2025

#️⃣연관된 이슈

📝작업 내용

가게 메뉴 업로드인데, 가게 이미지 업로드 api를 쏘고 있어 변경합니다.

메뉴 수정시 이미지 관련 이슈가 발생해서 백엔드와 상의 중 api url이 잘못된걸 발견했네요

서버 -> 클라이언트로 넘겨줄 때 파일 이름에서 확장자가 빠져 넘어와 여태 메뉴 수정시 400터지고 있었습니다..
백엔드측에서 확장자 검증을 따로 안하기로 했습니다.
해당 pr과는 상관 없는 이슈였기는 합니다!

스크린샷 (선택)

💬리뷰 요구사항(선택)

@l-lyun l-lyun requested a review from 99mini July 17, 2025 11:06
@l-lyun l-lyun self-assigned this Jul 17, 2025
Copy link
Member

@99mini 99mini left a comment

Choose a reason for hiding this comment

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

👍

@l-lyun l-lyun merged commit 53244b1 into main Jul 17, 2025
1 check passed
@l-lyun l-lyun deleted the fix/menu-image-upload-api-url branch July 17, 2025 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants