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

댓글 정보 조회 오류 수정 #242

Open
wants to merge 239 commits into
base: main
Choose a base branch
from
Open

댓글 정보 조회 오류 수정 #242

wants to merge 239 commits into from

Conversation

philos1234
Copy link
Member

About

댓글 정보 조회 누락 수정
#241

Description

  • UserInfoAdder 부분 수정
  • @GeneratedValue 삭제

Result

스웨거로 확인완료

Ref

#241

kimyouknow and others added 30 commits April 6, 2022 22:32
* Docs: 리드미 파일 수정

* Feat: test
Test : jwtService 테스트 코드 작성
* Feat: Newpost index 4번째 줄 주석처리

* Feat: SignUp 수정
* Chore: MarkdownEditor 컴포넌트 console 및 주석 삭제

* Refactor:   회원가입 form 양식 수정

기술 스킬: select 태그로 변경
useCallback 함수 추가

* Refactor: 선택한 기술 표시

* Refactor: signup test 데이터 삭제

* Chore: 회원가입 input validation 관련 주석 작성

* Feat: 회원가입에 희망작업기간 input추가

select 옵션 가능

* Refactor: 로그인한 유저 default 이미지 utils/cookie.js에서 관리

- loginNav.jsx에서 관리하던 default profile image 로직을 옮김

* Refactor: signUp/index.jsx useInput으로 리팩토링
* Refactor: login api response의 status관리

login 요청 결과로 받는 status로직 분해할당

* Refactor: skill옵션 데이터와 희망기간 데이터 contant파일로 이동

* Feat: 팀 게시글 생성 api 연결
* Refactor: setDefaultProfileImage 로직 constant로 이동

* Feat: useFileUploader.js 구현

- input 값이 바뀌면 file을 서버로 제출

* Chore: .env파일 .gitignore에 추가

* Feat: s3 이미지 서버 업로드 구현

* Chore: newPost 삭제

* Refactor: 로그인 유지 변경

- Authorization이 아닌 userInfo  쿠키 기준으로 로그인 유지

* Refactor: Callback page삭제

* Chore: Loader컴포넌트 src/pages -> src/components

* Feat: NewTeamPost에 이미지 업로더 구현

* Chore: 업로드한 이미지 콘솔출력
Team API URL 수정, 기술스택 형식 수정 #27 # 29
kimyouknow and others added 23 commits November 10, 2022 23:19
* Story: teampost, userpost 생성

* Fix: Image컴포넌트 src기본값 null로 변경

* Fix: ProfileImage컴포넌트 alt, src 기본값 추가

* Fix: Team Detail mock데이터 수정

* Fix: teamDetailParser 중 techSkills에서 skillStackParser제거

* Fix: teamDetailParser에서 writerInfo값 수정

* Fix: userDetailParser에서 techSkills기본값제거

* Design: 팀 포스트 상세페이지 css 수정

* Fix: 유저 상세 페이지 프로필 이미지 컴포넌트 수정

기본 이미지 컴포넌트 -> 프로필 이미지 컴포넌트
* Feat: useSetInterval

* Feat: useSetTimout

* Feat: OAuthFail페이지

* Feat: OAuth전용 callbac 페이지 라우팅

OAuthCallback

* Fix: 소셜로그인 컴포넌트 이미지 및 페이지 이동함수 수정

* Chore: oauth관련 최초 접속 url수정

* Chore: query-string 설치

* Feat: useHandleLogin

* Fix: 로그인페이지에 useHandleLogin 적용

* Feat: OAuthCallback
* Chore: hoc router 코드 컨벤션 수정

* Fix: 이메일중복체크, 닉네임중복체크 mock response수정

* Fix: 필수정보입력페이지 라우팅 Path 수정

* Feat: 필수정보입력 이동 path url 객체 생성

* Feat: Public 라우터 모음

* Feat: Private라우터 모음

* Feat: 기타 라우터 모음

* Feat: 필수정보입력페이지 nested라우팅 모음

* Feat: 로그인 페이지 nested라우팅 모음

* chore: /* 라우팅 변수명 변경 notfound -> all

* Refactor: 라우터 App.jsx에서 분리
* Chore: 기술스택 이미지 전체 assets폴더에 넣기

* Refactor: 기술스택 배열 constant파일 별도로 분리

* Refactor: TechSkills 컴포넌트 스토리북의 기술배열 수정

* Refactor: TechStackSelectInput에서 데이터 요청이 아닌 assets폴더의 기술목록 배열사용
* Fix: 팀소속여부 선택하는 필수정보입력페이지에 팀 소속여부 컴포넌트 적용

* Refactor:  OAuthCallback에 토스트 알람 쓸 수 있게 ToastNotificationProvider 적용

* Fix: 유저 mocking데이터 수정

* Feat: 기술스택관련 유틸 함수

getTechSkillsWithCategory
getRandomTechSkills

* Fix: 유저 카드 관련 css 수정

* Fix: 팀 Mock데이터 수정

* Chore: 팀 디테일 mock 데이터 수정

* Chore: 유저 디테일 mock 데이터 수정

* Refactor: layout 컴포넌트 헤더 유무로 구분

* Fix: 팀 게시글 페이지 css 수정

* Fix: WithInfiniteScroll에 callback페이지 적용

* Story: useBoard 스토리북 추가
* oauth 작업

* Feat: updateEssentialInfo에 job입력 추가

* Feat:(fe) 유저필수정보입력 기본값추가

* Fix(fe): Oauth 이후 최초로그인인지여부 string -> bool으로 수정

* UserEssential extract 수정

* Fix: 유저정보 api요청 실패 메시지 표시 후 로그인 페이지로 넘어가게 수정

delay 2초

* userInfoDto 및 UserDto 수정

Co-authored-by: yunho <[email protected]>
* Fix: useIntersect hooks 범용성 있게 수정

isIntersecting할 때 실행할 함수를 callback으로 넘겨 받기

* Refactor: WithInfiniteScroll 버그 수정

- 첫 렌더링 시 lastpage = 0인 상태로 두 번 api 요청하는 버그 수정
- intersection obsever api가 동작하지 않는 버그 수정정

* Feat: 유저보드 및 팀보드가 비어 있을때 표시할 메시지

* Refactor: 유저 및 팀보드에 수정한 WithInfiniteScroll 반영

* Feat: 유저 및 팀보드가 비어있을 때 유도 동작 표시

유저 -> 회원가입페이지로 유도
팀 -> 팀  생성 페이지로 유도

* Story: 유저.팀 빈 페이지 추가

* Refactor: refetcher이후 다시 실행되지 않는 버그 수정

ref를 살려두기

* Fix: 에러 상황일 때 추가 요청 방지

* Fix: callback 함수 중복 요청 방지

callback함수가 실행중일 때는 isIntersecting여도 다시 실행되지 않게 수정
* Fix: 무한스크롤 추가 요청 방지 로직

* Fix: 팀소속여부 체크 박스 옵션 추가(선택안되게 하는 옵션)

* Style: 메인레이아웃 상단 공백추가

* Fix: 유저 스키마 수정 (useInfo제거)

* Style: 유저디테일페이지 스타일 수정

- 이미지, 이름, 직업 좌측으로 이동

* Fix: useDropdown hooks 수정

- handleclickoustside 동작수정
- handleClickdropdownTrigger 추가

* Feat: PostInfo컴포넌트

- postId, 조회수, 좋아요수, 댓글수

* Style: 유저프로필수정페이지 이미지 스타일 수정

* Fix: 팀 포스트에서 postinfo 컴포넌트 분리

* Fix: 유저디테일 페이지에서 postinfo 컴포넌트 적용

* Fix: 팀파서에 readCnt추가

* Feat: 메인로고 변경

* Fix: SelectInput 화살표버튼 클릭 함수 버그 수정

* Style: 팀 생성 페이지 중앙 정렬

* Style: 팀 디테일 페이지 중앙 정렬

* Style: 내가 작성한 글 목록 중앙 정렬

* Style: NotAllow페이지

* Style: Notfound페이지

* Story: OAuthFail 페이지
* feat: main font를 제외한 나머지부분 완료 + route 완료

* style: 엔터추가

* refactor: 배경이미지 비율 맞추기를 제외한 나머지 텍스트 css 적용 완료

* style: MainContainer와 Section1구분

* style: Section2 구분

* style: Section3 구분

* style: Sectiton4 구분

* style: Section5 구분

* style: Section5 수정 후 Card와 Icon 주석으로 구분

* style: Section8 구분

* style: Bottom Box 구분

* 전체적인 Section top %값 조정

* refactor: absolute 삭제(relative로)

* refactor: naming 수정 + map 활용하여 반복되는 요소 수정

* style: 사용되지 않는 import문 삭제

* refactor: navigate 완료

* refacter: 개발자 이름 목록 map으로 변경

* refactor: theme 네이밍 수정 (숫자 들어간 부분)

* refactor: 마지막 부분 이름 map으로 처리

* refactor: component가 없는 배열들만 constant 파일 생성 후 이동

* refactor: text 움직이는 거 수정. width를 px값으로 변경

* Refactor: 메인페이지에서 cardText, links를 constant로 분리

* refactor: mainlayout router element 개별 관리

* feat: main에 gnb 추가 후 toastNotification추가

Co-authored-by: Wang HoEun <[email protected]>
Co-authored-by: yunho <[email protected]>
* feat: wavy animation + Ghost text + ScrollDown Indicator

* feat: GNB HideScrollOnScrollDown 적용

* feat: pulseEffect + CardEffect animation 적용

* style: bottom box에서 이메일과 이름 수정

* refactor: 리뷰 반영 하여 wavy map으로 수정 + 상수들 constant로 이동

Co-authored-by: Wang HoEun <[email protected]>
@GeneratedValue 삭제
@kimyouknow
Copy link
Contributor

아래와 같은 에러가 발생합니다. @philos1234

  1. POST​/api​/user​/comment 실행시 500에러 발생

spring 콘솔

스크린샷 2022-12-31 오후 3 40 01

  1. 댓글 삭제 이후 아래와 같은 에러 발생

예를 들어, A유저로 로그인한 후 게시글에 댓글을 작성 -> A유저로 로그인한 상태에서 본인이 작성한 댓글 삭제 -> A유저 관련 모든 동작에서 에러 발생합니다.

  • get /api/user/:userID : 유저 게시글 조회
  • get /api/user/login: 로그인

스크린샷 2022-12-31 오후 3 43 37

@philos1234
Copy link
Member Author

philos1234 commented Jan 4, 2023

@kimyouknow 혹시 정확히 어떤 요청들을 했는지 알수있나요 i
요청시 날렸던 body까지?
저는 1번과 2번 경우에 모두 정상적으로 작동합니다
1번의 경우는 조건에 맞게 제대로 요청을 날렸는지 확인 부탁하며
2번의 경우는 자기 댓글을 삭제한 경우가 아니라 유저를 삭제한게 아닌지..??

@kimyouknow
Copy link
Contributor

kimyouknow commented Jan 5, 2023

1번과 2번 모두 각각 디비를 초기화한 후 다시 실행했습니다.

  1. OAuth 에러

구글 및 깃허브 소셜 로그인시 아래와 같은 에러가 발생합니다. 에러메세지의 원인이 아래와 같아 댓글 관련 pr에 수정요청합니다.

스크린샷 2023-01-05 오후 3 01 35

org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: com.projectmatching.app.domain.user.entity.User.userComments, could not initialize proxy - no Session
	at org.hibernate.collection.internal.AbstractPersistentCollection.throwLazyInitializationException(AbstractPersistentCollection.java:606) ~[hibernate-core-5.4.32.Final.jar:5.4.32.Final]
	at org.hibernate.collection.internal.AbstractPersistentCollection.withTemporarySessionIfNeeded(AbstractPersistentCollection.java:218) ~[hibernate-core-5.4.32.Final.jar:5.4.32.Final]
	at org.hibernate.collection.internal.AbstractPersistentCollection.readSize(AbstractPersistentCollection.java:162) ~[hibernate-core-5.4.32.Final.jar:5.4.32.Final]
	at org.hibernate.collection.internal.PersistentSet.size(PersistentSet.java:168) ~[hibernate-core-5.4.32.Final.jar:5.4.32.Final]
	at com.projectmatching.app.domain.user.dto.UserDto.of(`UserDto.java:61`) ~[main/:na]

UserDto.java:61

userDto.commentCnt = user.getUserComments().size();

에러 메세지 이후 db 상황

스크린샷 2023-01-05 오후 3 30 14

위와 같은 에러메시지와 함께 spring이 동작하지 않지만 db에 관련 정보는 저장됩니다.

  1. GET: ​/api​/user​/comment: commendId 실행시 500에러 발생

언급해주신 것처럼 POST​ /api​/user​/commen: commenId 는 정삭적으로 동작합니다. 이후 곧바로 GET POST​/api​/user​/comment/:commendId 할 때 500에러가 발생합니다.

login user id: 9880448693
GET​/api​/user​/comment/5387141186 # -> 정상
POST​/api​/user​/comment # -> 정상
GET​/api​/user​/comment/5387141186  # -> 500
GET​/api​/user​/comment/5387141186 # -> 이후 다시 요청하면 계속 500

post 요청 body

{ 
userId: "5387141186", parentId: 0, secret: false, content: "하하하"
},

스크린샷 2023-01-05 오후 3 28 45

에러 메세지

eq(null) is not allowed. Use isNull() instead; nested exception is java.lang.IllegalArgumentException: eq(null) is not allowed. Use isNull() instead

Base automatically changed from develop to main January 10, 2023 07:55
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.

4 participants