-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FE] 리뷰 작성 기능 구현 #13
Comments
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
알라딘 api 버전을 20070901 -> 20131101 로 업데이트하면서 응답 포맷이 자바스크립트 객체로 변경됨 - 서버단에서 json 변환 작업 생략 - 클라이언트단 reducer에서 JSON -> JS 객체로 변환해주는 작업 생략
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
1. Modal 내용 렌더링 로직 개선 Modal을 호출하는 쪽에서 template 및 template에 들어갈 content를 props로 전달하고 Modal 내부에서 template에 content를 주입하도록 변경 -> Modal 내부에서 비동기 요청하여 응답 결과를 template의 content로 전달할 수 있도록 함 (props의 apiCallback를 호출하여 비동기 요청) 2. Modal 내부에서 feedback Modal 렌더링할 수 있도록 기능 구현 feedback Modal의 경우 clickOutside 기능 비활성화 (useClickOutside 훅 내부에서 클릭된 엘리먼트로부터 'click-outside-disabled' id값을 가진 엘리먼트가 탐지될 경우 clickOutside 기능 실행 취소) 3. feedback Modal에 사용될 FeedbackTemplate 추가
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
이미 리뷰를 작성한 도서를 클릭시 리뷰작성 모달과 함께 피드백 모달 제공 -> 피드백 모달의 '확인' 버튼 클릭시 리뷰작성 화면 제거 후 리뷰상세 화면 제공 <문제> 리뷰상세 모달은 리뷰 작성 모달의 하위 컴포넌트이기 때문에 리뷰작성 모달을 useModal 훅을 사용하여 unmount하게 될 경우 리뷰상세 modal도 함께 unmount됨 <해결> - Modal type (modalFor props)에 따라 각각의 modal root (subWrapper)를 생성 - 리뷰작성 modal root(DOM Element)의 style.display 값을 'none'으로 변경하여 리뷰작성 화면 숨김 - Modal 컴포넌트 내부에서 모든 modal root의 display 값을 'block'으로 원복
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
- Modal Header 스타일 수정 - FeedbackTemplate 스타일 추가
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
type은 redux 상태관리 및 컴포넌트에서 공통적으로 사용됨 -> /feature 내부에서 관리하던 redux용 types를 /common/types에서 일괄적으로 관리하도록 변경
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
reesekimm
added a commit
that referenced
this issue
Oct 7, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
설명
리뷰 작성 기능 구현
구현 상세
The text was updated successfully, but these errors were encountered: