Week5 PR Develop to master #342
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
feature: hashtag DB query api with naive test code
Hot fix/host app oauth
하드코딩되어있던 graphql 쿼리를 lib 의 getQuestionsByEventCodeAndGuestId 함수로 분리함
question.graphql 에서 QuestionId: Int 추가 isStared: Boolean 추가
단순 local 에서의 상태 변경이었던 star 버튼을 이벤트를 발생시켜 isStared 속성을 변경하도록 구현.
Candidate => Candidates
개발 환경에서 host-page default token을 추가.
pollType은 3가지임 - 0: N지선다(text) - 1: N지선다(date) - 2: 별점 makePollDummy()에서 생성된 pollType과 동일하게 makeCandadateDummy()가 생성되도록 수정함 O/X는 N지선다중 text 방식의 보기의 갯수가 2인 것임 pollType이 2일때(별점)는 보기의 갯수가 별점 총수와 동일하도록 함
env: 배포시 DB migration 추가
env: 최상위 폴더에서 yarn start 로 front, back 둘다 dev mode 로 실행하는 yarn run script 추가
migration * selection table 삭제 * voter 테이블 삭제 * vote 테이블 추가 seeder * voter 더미 데이터 seeder 삭제 * selection 더미 데이터 seeder 삭제 models * candidate 테이블 모델 관계 업데이트 * guest 테이블 모델 관계 업데이트 * voter 테이블 모델 삭제 * selection 테이블 모델 삭제 * vote 테이블 모델 추가
graph ql auth 관련해서 기존 passport-jwt 사용을 jwt-verify로 변경. graphql 미들웨어를 추가하여 토큰값에 따라 인증허가 로직 구현. waring: 미들웨어에서 인증허가는 들어온 토큰 값에 따라 사용자를 구분하고 실제로 허가 부분은 리졸버단에서 구현해야함.
* event table 컬럼 이름 변경 code -> eventCode * class 형식으로 된 event query를 함수로 분해
candadate -> condidate dummy.js -> pollDummyData.js
getQuestionsInEvent -> getQuestionsByEventCodeAndGuestId
saveContext->authenticate
fix: DB poll dummy data 생성방식 수정
fix: guest, candidate 연관 관계 수정
Fix/db object attribute name
graph ql 인증 로직 수정 및 dev 환경에서도 가능하도록 함
마지막에 '/'들어가 경로가 꺠져 수정함
hotfix: main-app 에서 guest app 으로 리다이렉트 주소
Feature/host app move question
Host-app/Poll 기능 추가 및 수정
1. guest info를 가져오는 graphql schema와 resolver를 추가하여 데이터를 불러올 수 있도록 구현 2. guest front end 부분에서 초기데이터를 불러올 수 있도록 cookie 값에 저장된 jwt를 복화하여 token으로 건네는 작업 구현 3. host-page event setting modal 부분 시작시간 date type error 수정
feat: guest app routing 게스트 방 식별 구현
updateQuestionById 에서 content 이외의 정보를 수정할 수 있도록 쿼리 수정.
…into feature/host-app-move-question
useContext 추가하여 하드코딩으로 박혀있던 부분 수정
현재 라우팅 정보에 따라 각각의 방으로 입장 가능하여 적절한 데이터를 가져오도록 수정. jwt의 payload에 따라 사용자 정보를 판별하고 이에 따라 각각의 방에 맞게 라우팅시킴 이벤트 생성 시 즉각적으로 질문 페이지가 렌더되도록 수정
유저 정보에 따른 방입장
일부 하드코딩된 eventId 가 남아있어 moderation 이 제대로 동작하지 않는 문제를 수정. guestName 이 DB 에 빈칸으로 들어가던 문제 수정
글을 쓸 때 guestName 을 update 하는 쿼리를 추가하여 update된 guestName 을 DB 에 넣도록 수정
임시 함수인 JSONNestJoin2 를 사용하여 동일 guestId 에 대한 question 이 filter 되던것 수정
develop 에서 merge
host link 생성 시 dev와 prod에 따라 다른 url을 추기 위해 config 추가
fix: question 의 guestName이 DB 에 제대로 들어가지 않는 문제 수정
enhance : add url for dev,prod config
useContext(GuestContext) 를 이용하여 가져옴
guest-app 에서 초기 데이터를 전부 받아와서 moderation 된 데이터도 받아오는 문제 수정
Feature/poll: useContext를 이용하여 Poll 데이타를 가져오는 기능
fix: guest-app moderation question 새로고침시 뜨는 문제 해결
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.
PR code: week5/nicky