Skip to content

HotFix: 일부 스터디 API 기능 문제 해결#120

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

HotFix: 일부 스터디 API 기능 문제 해결#120
Ruthgyeul merged 2 commits intomasterfrom
develop

Conversation

@Ruthgyeul
Copy link
Member

#️⃣연관된 이슈

#102

📝작업 내용

일부 스터디 API 기능 문제 해결

@Ruthgyeul Ruthgyeul requested a review from Copilot May 8, 2025 16:16
@Ruthgyeul Ruthgyeul self-assigned this May 8, 2025
@Ruthgyeul Ruthgyeul added 💻 MASTER Master PR 요청 HotFix 긴급히 오류를 고칠 때 labels May 8, 2025
@github-advanced-security
Copy link
Contributor

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

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 issues in the study API functionality.

  • In ReviewApplication.jsx, the changes streamline event handler creation by importing and using useCallback directly.
  • In page.jsx, the S3 key string for image uploads was modified.

Reviewed Changes

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

File Description
src/components/study/dashboard/admin/detail/ReviewApplication.jsx Updated event handler callback to use the imported useCallback and added local storage removal and state updates before page reload.
src/app/study/create/page.jsx Modified the S3 key string from "/study/" to "study" for the image upload process.
Comments suppressed due to low confidence (1)

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

  • The change from '/study/' to 'study' for the S3 key may affect the file path structure expected by the API. Verify that the new key format correctly meets the S3 upload requirements.
const s3key = `study`;

@Ruthgyeul Ruthgyeul merged commit 8d89a57 into master May 8, 2025
5 checks passed
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