Skip to content

HotFix: 이미지 업로드 API 오타 수정#121

Merged
Ruthgyeul merged 2 commits intomasterfrom
develop
May 8, 2025
Merged

HotFix: 이미지 업로드 API 오타 수정#121
Ruthgyeul merged 2 commits intomasterfrom
develop

Conversation

@Ruthgyeul
Copy link
Member

#️⃣연관된 이슈

#102

📝작업 내용

이미지 업로드 API 오타 수정

@Ruthgyeul Ruthgyeul requested a review from Copilot May 8, 2025 16:48
@Ruthgyeul Ruthgyeul self-assigned this May 8, 2025
@Ruthgyeul Ruthgyeul added 💻 MASTER Master PR 요청 HotFix 긴급히 오류를 고칠 때 labels May 8, 2025
@Ruthgyeul Ruthgyeul merged commit 714e5c3 into master May 8, 2025
5 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This hotfix addresses a typo in the image upload API by correcting the property name for retrieving the S3 key. Additionally, there is an unrelated update to mobile PWA meta tags in the layout file.

  • Fix in image upload API: Changed "s3key" to "s3Key" to correctly capture the S3 response field.
  • Update in meta tags: Replaced Apple-specific meta tags with generic mobile-web-app tags.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/app/study/create/page.jsx Corrected the typo in the S3 key property to ensure successful image upload processing.
src/app/layout.js Updated PWA meta tags which may impact Apple device compatibility.
Comments suppressed due to low confidence (1)

src/app/layout.js:120

  • Replacing Apple-specific meta tags with generic mobile-web-app tags may break PWA support on Apple devices; consider retaining the original Apple meta tags or including them alongside the new tags if iOS compatibility is required.
<meta name="mobile-web-app-capable" content="yes"/>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

HotFix 긴급히 오류를 고칠 때 💻 MASTER Master PR 요청

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants