Week6 PR Develop to master #424
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
js 파일들을 guest-app에서 가져온 것이라 Poll 컴포넌트에 클릭함에 의해 상태가 변하는 로직이 있었는데, host-app에서는 클릭에 의한 상태 변화가 없으므로 제거함
10개로 제한했던 data fetching 을 정상적으로 받아오도록 변경
쿠기값에 여부에 따라 기존의 연결된 세션으로 이동할 수 있는 네비게이션 UI 추가
moveQuestion 기능을 구현하고 DB 에 update 되어 새로고침시에도 적용
# why QuestionId, GuestId를 가지는 행을 지우기 위해 필요함 # what i did deleteLikeBy 함수 추가
아카이브 기능을 DB 와 연동하여 새로고침시 적용되도록 변경
# why 필수 기능 # what i did DB 에 접근해 해당하는 guestId와 questionId 정보를 삽입한뒤, namespace내의 전체 socket에 이벤트 전달함
# why 필수 기능 # what i did DB 에 접근해 해당하는 guestId, questionId와 일치하는 행을 삭제후, namespace내의 전체 socket에 이벤트 전달함
# why 좋아요 버튼에 대한 상태를 조상 컨테이너에서 사용함 # what i did LikeButton 컴포넌트에 useLikeButton 대신 props를 전달함
로딩시 스켈레톤을 출력하도록 변경
host-app 각각의 카드 컴포넌트에서 more 버튼을 눌러 질문을 삭제할 수 있음
# why 필수 기능 # what i did * 기존 LikeButton 컴포넌트에 socket.io emit 처리 추가 * QuestionsReducer 에 onQuestionLike 핸들러추가 * QuestionsReducer 에 onUndoQuestionLike 핸들러추가 * 가독성을 위해 QuestionsReducer의 핸들러 맵핑 객체의 함수 분리 -> QuestionsReducer에 핸들러에 현재 state 와 data를 인자도 넣도록 수정함 * QuestionContainer의 useSocket과 데이터 로딩하는 useEffect 부분을 함수로 분리
# why 이모지 추가 삭제 기능의 서버와 연동을 위해 # what i did * EmojiInsertButton 을 다른 컴포넌트로 분리 * 더미 데이터 삭제 * onVote 콜백 함수의 이름을 onClickEmoji로 변경 * onSelect 콜백함수의 이름을 onAddEmoji로 변경 * EmojiInstance의 voted props의 이름을 didIPicked로 변경 * 부모 컴포넌트로부터 emoji정보를 받을 수 있도록 props를 내려줌
# why 빈 배열에 추가되도록 해야함
# why 질문별 생성된 이모지 순서가 필요함 # what i did * DB query 에 가장먼저 추가된 시간을 group by 하도록 변경 * emoji scheme에 creactedAt 추가
# what i did * guest-app 에 누락된속성 추가 * socket.io hander에 누락된 속성 추가
# why EventId컬럼 추가가 누락됨
# why 필수 기능 # what i did ## 실행 흐름 각 이모지별 클릭시 콜백으로 emoji의 name 인자로 전달함, 콜백에서 socket.io로 필요한 데이터를 서버로 송신 서버에서 받은 데이터를 DB에 넣은 후 해당 이벤트의 guset와 host에 emit guest는 해당 이벤트를 받았을때 reducer에 global데이터를 담아 dispatch 함 reducer에서 받은 데이터로 questions 상태를 업데이트 ## etc * EmojiInstance.js의 불필요한 props 제거
# why 필수 기능
# what i did * QuestionsReducer에 remove Question emoji 핸들러 추가 * 새로운 이모지 버튼 클릭시 추가되는 이벤트에 대한 처리를 기존 reducer의 addQuestionEmoji 에서 처리하도록 구현 * 기존 onClickEmoji의 로직을 pickEmoji함수로 분리 * onSelectOfEmojiPicker 에서 중복된 이모지에 대한 validation 처리 * QuestionContainer에서 useSocket `questionEmoji/remove` 이벤트 처리
# why 시간 순정렬시 비교 대상이 string 타입의 숫자이여야하는데 새로 추가된 질문은 number 타입이다 # what i did * socket.io 서버 측에서 새로 생성시 string 타입으로 전환시킴 * guest-app에서 보내는 getNewQuestion의 불필요한 속성 제거
# what i did * questionLike/create socket.io이벤트 시 reducer이벤트 이름 수정 * reducer 이벤트 타입을 동사형 먼저 나오도록 수정 * add newQuestion 시 글로벌 데이터를 주입함
# what i did * BottomPaddingBox 컴포넌트를 다른 파일로 분리함 * BottomPaddingBox 의 이름을 PaddingArea로 변경 * useMyQuery 함수를 인라인화
# why 로드하기전 socket.io를 통해 들어온 질문이 사라질 수 있음
별점매기기를 위한 ratingValue, rated 필드 추가
* libs의 하위 디렉토리의 파일을 babel 빌드에서 누락됨
hotfix: backend build 스크립트 수정
* redis의 하위 디렉토리의 파일을 babel 빌드에서 누락됨
hotfix: backend build 스크립트 수정
guest-app 상단고정 기능 구현. warnings: sync 기능은 아직 구현되지 않음
guest 페이지 진입 시 쿠키값을 가지고 있을 때 해당 쿠키값에 종속되어 다른 이벤트로 입장해도 기존의 이벤트로만 라우팅되는 문제가 발생 이를 해결하기 위해 guestAuthenticate 미들웨어에 comparePathToCookie 함수를 추가하여 파라미터로 받은 query를 검사하고 이벤트 id와 같은지 확인 후 다르면 쿠키값을 재발급하는 방식으로 문제 해결 호스트 페이지에서 이벤트 생성 시 해시태그를 집어넣고 이벤트를 생성할때 해시태그에 대한 정보는 들어가지 않아 버그가 발생 이벤트 생성과 동시에 컨텍스트에 저장된 이벤트에 해시태그 목록도 함께 업데이트하는 방식으로 버그 해결
host-app과 guest-app 사이의 상단고정 sync 기능을 구현
guest-app 에서 최초 data를 load 하고 buildQuestions 할 때 active인 상태만 filter 해서 최초 active 상태가 아닌 경우 host-app 에서 질문을 restore 해도 guest app 에 나타나지 않는 문제가 발생하여 이를 수정함
socket.io 서버의 req 메세지명의 변경으로 인해 host-app 에 data 가 바로 추가되지 않는 문제 fix
* redis client config 수정 * docker -production.yml 수정
hot-fix: 질문 host-app 에 추가 안되는 문제 fix
chore: vaagle 로고 대신 "바글바글" text로 변경
* docker pm2에서 파일로깅이 안됨, 일단 제거함
hotfix: deploy fix
이벤트 네임과 이벤트 유효시간 변경사항을 즉각적으로 반영하기 위해 추가
fix: error log 추가
fix: style 수정
guest-app 에서 isStared 를 처리할 때 이전에 켜져있던 isStared 를 false 로 변경하지 않아 중복해서 상단고정 되던 문제해결
fix: 상단고정 guest-app 에서 중복해서 나타는 현상 제거
hotfix: host 에서 standby poll 생성 후 refresh 하면 죽는 문제 수정 팀원 확인 하에 hot-fix merge
prod.config 파일 www. 누락으로 인한 에러 ;;
fix: guest api link route 에러
지난 hotfix와 동일하게 pollDate에 null 값이 들어가 있는데 여기에 값이 있다고 가정하고 처리하는 루틴이 있어서 수정함 추후 DB를 보완하겠음
hotfix: host 에서 open한 poll이 guest에서 보여질때 죽는 문제 수정
|
팀원들의 동의 하에 master로 merge |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.