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

Feature/#74/display user connection status #75

Merged

Conversation

dohyeondol1
Copy link
Collaborator

@dohyeondol1 dohyeondol1 commented Sep 7, 2024

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가 ✨
  • 기능 삭제 🔥
  • 버그 수정 🐛
  • 코드 형태 개선 🎨
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트 🔨

변경 사항

친구 목록에서 다른 유저들의 접속 상태를 확인할 수 있도록 '온라인', '오프라인'이 표시되는 기능을 추가하였습니다.

이슈 사항

  • 브랜치에 behind되어있던 커밋들도 함께 머지합니다. 제가 수정한 파일은 friend.js입니다~!
  • DB에서 표시되는 온라인 유저의 수와 웹에서 표시되는 온라인 유저의 수가 같아야 합니다.

To reviewer

최소한의 기능만 만들어서 올렸습니다만, 형식을 조금 더 다듬어 다시 PR올리겠습니다~!

os-hoon and others added 20 commits August 11, 2024 19:41
mybatis -> jpa로 변경 (User 컴포넌트)
카카오 로그인 부분 추가
SecurityConfig에 경로 추가해줘서 카카오톡 소셜 로그인으로 리다이렉트 안되게 방지
- 인증실패 시 로그인페이지로 리다이렉션이 아닌 401코드 반환.
- 일반로그인을 form로그인으로 변경.
user테이블에 시큐리티 로그인때문에 만들었던 id와 동일한 역할을 하는 username은 없어도 정상 작동함.
카카오 로그아웃을 위한 엑세스토큰을 세션에 보관하고 디버깅용 코드는 삭제함
[feature] 소셜로그인, 일반로그인 수정
post 컴포넌트
mybatis -> jpa 방식으로 수정
postComment 관련 컴포넌트에서 post_comment 테이블에서 복합키 설정 안되어있던 문제 해결, 게시물 삭제시 댓글은 남아있던 문제 함께 해결 완료, userhome 관련 컴포넌트 mybatis ->jpa 방식으로 수정
mybatis->jpa 수정
* [fix] Cors설정 헤더추가

* [fix] 세션설정수정

* [fix] 세션설정 수정 import문 추가

* [fix] 마리아DB 버전 업데이트

* [fix]폼로그인 권한,cors수정

* [fix] cors수정

* [fix] 스프링시큐리티로 cors수정

---------

Co-authored-by: Siyun Park <[email protected]>
jpa -> mybatis로 다시 수정
User컴포넌트
jpa-> mybatis로 되돌림
로그인 리다이렉션 경로 수정, 회원가입시 비밀번호 암호화 방식 수정
form로그인 에러 원인을 찾기 위한 수정
[feature] 카카오 소셜로그인 추가, form로그인 수정
- state 값이 1이면 온라인, 0이면 오프라인으로 표시하였습니다.
@dohyeondol1 dohyeondol1 added 도현석 프론트엔드 기능 labels Sep 7, 2024
@dohyeondol1 dohyeondol1 requested a review from os-hoon September 7, 2024 08:36
@dohyeondol1 dohyeondol1 self-assigned this Sep 7, 2024
@dohyeondol1 dohyeondol1 merged commit 5a45916 into feature/#61/front_code Sep 7, 2024
@dohyeondol1 dohyeondol1 linked an issue Sep 7, 2024 that may be closed by this pull request
2 tasks
@dohyeondol1 dohyeondol1 deleted the feature/#74/display_user_connection_status branch September 10, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 도현석 프론트엔드
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feature] 유저 접속 현황 표시하기
3 participants