Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix : 정적 파일 경로 설정 오류(학생 이메일 비밀번호 수정) #1101

Merged
merged 2 commits into from
Nov 29, 2024

Conversation

duehee
Copy link
Contributor

@duehee duehee commented Nov 29, 2024

🔥 연관 이슈

🚀 작업 내용

  1. 학생이 이메일로 비밀번호를 변경 할 수 없었던 오류를 수정했습니다.
image

💬 리뷰 중점사항

현재 정적파일을 사용하는 곳은 학생이 이메일을 통하여 비밀번호를 수정하는 로직이 있습니다. 예전 존재하지 않는 API 응답 커스텀하기 관련 yml 파일 수정(add-mappings = false)이 있었고, 이 과정에서 정적 파일의 경로를 찾을 수 없는 오류가 있었습니다.

WebConfig 중 addResourceHandlers를 추가하여 /static으로 통하는 정적 파일의 경로를 지정해줬습니다.
앞으로 정적 파일의 관리가 필요하다면 /static에 넣고 사용하시면 됩니다.

ㅜㅜ 빙빙 돌아서 오류는 해결했습니다.. 도와주신 분들 감사합니다.

@duehee duehee added 버그 정상적으로 동작하지 않는 문제상황입니다. Team User 유저 팀에서 작업할 이슈입니다 labels Nov 29, 2024
@duehee duehee self-assigned this Nov 29, 2024
Copy link
Contributor

@Soundbar91 Soundbar91 left a comment

Choose a reason for hiding this comment

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

근장하시면서 오류 해결,,
고생하셨습니다 💯

Copy link

Unit Test Results

339 tests   338 ✔️  1m 29s ⏱️
  41 suites      1 💤
  41 files        0

Results for commit 02a4fc4.

Copy link
Contributor

@seongjae6751 seongjae6751 left a comment

Choose a reason for hiding this comment

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

고생 많았어요ㅜ

Copy link
Contributor

@kwoo28 kwoo28 left a comment

Choose a reason for hiding this comment

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

수고 많으셨어요!

Copy link
Collaborator

@daheeParkk daheeParkk left a comment

Choose a reason for hiding this comment

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

수고많으셨습니다..

@duehee duehee merged commit d78bb87 into develop Nov 29, 2024
5 checks passed
@duehee duehee deleted the fix/1100-student-change-password-by-email branch November 29, 2024 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team User 유저 팀에서 작업할 이슈입니다 버그 정상적으로 동작하지 않는 문제상황입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

정적 파일 로드 안 되는 오류
5 participants