-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Epic#8, Epic#10] 스피커, 리스너의 히스토리카드로 채널 입장 가능 #44
Comments
Merged
happydhKim
added a commit
that referenced
this issue
Nov 29, 2019
Why - 마이페이지로 이동할 수 있어야 한다. What - 헤더의 프로필 메뉴로 mypage로 이동 할 수 있도록 구현.
happydhKim
added a commit
that referenced
this issue
Nov 29, 2019
Why - 히스토리 카드를 눌렀을 때, 해당 채널로 이동해야 한다. What - 해당 채널 ID 값으로 카드 클릭 시 이동 되도록 구현.
happydhKim
added a commit
that referenced
this issue
Nov 29, 2019
Why - 리스너가 채널에 입장하면 히스토리 기록이 남아야 한다. What - 채널 정보를 가져오는 resolvers의 getChannel 메서드에서 마스터를 먼저 확인. - 마스터가 본인이 아니고, 히스토리 기능이 없다면 히스토리에 추가.
happydhKim
added a commit
that referenced
this issue
Nov 30, 2019
Why - 채널이 중복되는 현상 수정이 필요하다. - 히스토리 카드 간격 및 스타일 리펙토링이 필요하다. What - 헤더에서 프로필을 눌렀을 때, 안닫히던 현상 수정. - 히스토리 카드 간격 및 스타일 일관성있게 수정. - 채널이 중복되어 입력되는 현상 수정. ( 채널 체크시 함께 적용 )
happydhKim
added a commit
that referenced
this issue
Dec 1, 2019
Why - 리뷰 받은 사항 중 도메인별 resolver를 나눠야 하는 상황 발생. What - 도메인을 나누기 위해 resolvers/histories.js 에 addHistory를 추가. - history mutation을 추가. - 기존에 channel에 있던 history 관련 리팩토링.
happydhKim
added a commit
that referenced
this issue
Dec 1, 2019
Why - 리뷰에서 파라미터 리팩토링 요구를 처리해야 함. What - 기존 방식에서 destructuring으로 변경.
happydhKim
added a commit
that referenced
this issue
Dec 1, 2019
Why - 히스토리에 날짜를 보기 쉽게 표현해 줘야 한다. What - util에 dateParser함수 구현. - 히스토리 카드에 연동.
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
[Epic#8, Epic#10] 스피커, 리스너의 히스토리카드로 채널 입장 가능
완료 조건 ✅
관련 이슈 📎
레퍼런스 📚
The text was updated successfully, but these errors were encountered: