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.

Sign up for GitHub

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

Jump to bottom

fix: logout 에러 수정 #146

Merged
merged 1 commit into from
Mar 13, 2024
Merged

fix: logout 에러 수정 #146

merged 1 commit into from
Mar 13, 2024

Conversation

gitchannn
Copy link
Contributor

@gitchannn gitchannn commented Mar 13, 2024 •
edited
Loading

주요 변경사항

  • 로그아웃 시에 isLoggedIn을 FALSE로 바꾸는 로직을 추가하면서, 이후에 세션이 유효한지 찾을 때 isLoggedIn이 TRUE인 것중에 찾도록 해야하는데 깜빡해서 자꾸 2개 이상의 결과를 반환하는 에러가 나고 있었습니다! 수정 완료 cc. @JunhyeongPark-kr

관련 이슈

closes #

✔️ Squash & Merge 방식으로 병합해 주세요!

@gitchannn gitchannn self-assigned this Mar 13, 2024
@gitchannn gitchannn merged commit 8cee305 into main Mar 13, 2024
1 check passed
@gitchannn gitchannn deleted the fix/logout branch March 13, 2024 05:50
gitchannn added a commit that referenced this pull request Mar 13, 2024
@gitchannn
fix: logout 에러 수정 (#146)
5199ff7
gitchannn added a commit that referenced this pull request Mar 13, 2024
@gitchannn
feat: 한 아이디로 하루 10회 초과 점역 요청 시 예외 발생 (#142)
bc19b42 
* feat: 한 아이디당 10회씩 점역 가능하도록 한계 설정

* fix: 세션 아이디 파라미터로 검사하는 로직 제거 (#143)

* feat: Login ID에 unique 제약조건 설정 (#145)

* fix: logout 에러 수정 (#146)

* feat: 10회 초과 시 예외 발생
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gitchannn gitchannn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gitchannn