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: 오류사항 6건 수정 #589

Merged
merged 6 commits into from
Jun 11, 2024
Merged

fix: 오류사항 6건 수정 #589

merged 6 commits into from
Jun 11, 2024

Conversation

KimTom89
Copy link
Collaborator

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요. PR을 보내기 전에 모든 항목을 확인해야 합니다.

  • 동일한 업데이트/변경에 대한 다른 Pull Requests가 열려있는지 확인했습니다.
  • 테스트가 성공적으로 수행되었는지 확인했습니다.

PR 유형

어떤 유형의 PR인가요? (해당 항목에 모두 체크해주세요)

  • 버그 수정
  • 새로운 기능
  • UI/UX 개선
  • 문서내용 수정
  • 코드 의미에 영향을 주지 않는 변경사항 (오타, 서식 지정, 변수명 변경 등)
  • 코드 리팩토링 (버그 수정이나 기능 변경 없는 코드 변경)
  • 빌드 관련 변경
  • 테스트 코드 추가
  • 기타 (이유를 설명해주세요.)

변경 사항

1. 아이디 찾기 결과 페이지 > 비밀번호 찾기 버튼 링크 오류 수정

2. JWT 생성/만료일 계산 오류 수정

  • #543에서 JWT 라이브러리 변경으로 인한 오류

3. 회원 가입 > 여분필드 입력 가능하도록 수정

4. REST API > 모든 요청에서 OAuth2PasswordBearer(Optional)인증이 표시되는 오류 수정

  • 의존성 주입이 아닌 직접 회원정보를 조회

5. 1:1문의사항 오류 수정

  • 답변이 없는 문의글의 상세페이지에 들어갈 수 없는 오류 수정
  • 답변 작성 시 답변상태가 바뀌지 않는 오류 수정
  • 불필요한 권한체크 삭제

6. 설치 > admin_member_setup 함수 내부 오타 수정

  • Member().mb_id -> Member.mb_id

관련 이슈

기타 정보

KimTom89 added 6 commits June 5, 2024 11:10
- 답변이 없는 문의글의 상세페이지에 들어갈 수 없는 오류 수정
- 답변 작성 시 답변상태가 바뀌지 않는 오류 수정
- 불필요한 권한체크 삭제
- 현재 접속자 갱신 함수에서 의존성 주입이 아닌 직접 회원정보를 조회하도록 수정
- #543에서 JWT 라이브러리 변경으로 인해 내부 로직이 변경되어서 발생한 오류
@KimTom89 KimTom89 merged commit 9c97078 into master Jun 11, 2024
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.

1 participant