Skip to content

HotFix: fix fetching JSON data of API#122

Merged
Ruthgyeul merged 1 commit intomasterfrom
develop
May 8, 2025
Merged

HotFix: fix fetching JSON data of API#122
Ruthgyeul merged 1 commit intomasterfrom
develop

Conversation

@Ruthgyeul
Copy link
Member

#️⃣연관된 이슈

#102

📝작업 내용

fix fetching JSON data of API

@Ruthgyeul Ruthgyeul requested a review from Copilot May 8, 2025 18:05
@Ruthgyeul Ruthgyeul self-assigned this May 8, 2025
@Ruthgyeul Ruthgyeul added 💻 MASTER Master PR 요청 HotFix 긴급히 오류를 고칠 때 labels May 8, 2025
@Ruthgyeul Ruthgyeul merged commit 1648a0c 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 PR hotfixes the JSON data fetching issue when uploading images to S3. The changes update the extraction of the S3 key from the JSON response, modify the error handling strategy from throwing an exception to logging an error, and add a console output of the update form data for debugging purposes.

  • Updated S3 key extraction from resS3.data.s3Key to resS3.data.data.s3Key
  • Modified error handling: removed thrown error in favor of a console.error log
  • Added debug log for updateFormData
Comments suppressed due to low confidence (1)

src/app/study/create/page.jsx:83

  • Replacing a thrown error with a console.error may allow processing to continue with an undefined S3Key, leading to potential downstream issues. Consider reverting to throwing an error or implementing additional error handling to prevent further processing in this case.
if (!getS3Key) console.error("Error: uploaded image's S3Key Not Found");

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