Fix: 댓글 기능 추가(댓글 줄바꿈 표시, 비회원 댓글창 클릭시 로그인 모달, 답글 작성자 프로필 클릭 로직 추가), 인트로 페이지 영상 ios 자동재생 문제 테스트 #242
Conversation
This file contains 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
auto-assign
bot
requested review from
Eugene-A-01,
kanglocal,
ParkSohyunee and
seoyoung-min
|
@Nahyun-Kang is attempting to deploy a commit to the Eujin Ahn's projects Team on Vercel. A member of the Team first needs to authorize it. |
ParkSohyunee
approved these changes
May 17, 2024
| <video src={'/videos/video_section3.mp4'} autoPlay loop muted className={styles.video} /> | ||
| <video src={'/videos/video_section3.mp4'} autoPlay loop muted playsInline className={styles.video} /> |
There was a problem hiding this comment.
나현님 덕분에 비디오 인라인 재생시 playsInline 속성을 추가해야 한다는 점! 새롭게 배워갑니당👍👍
Comment on lines
118
to
122
| {isOn && ( | ||
| <Modal handleModalClose={handleSetOff} size="large"> | ||
| <LoginModal id="redirectedLoginBtn" /> | ||
| </Modal> | ||
| )} |
There was a problem hiding this comment.
나현님, 여기 댓글 로그인 모달 id는 사용자가 어디서 로그인 했는지를 데이터로 쌓기 위해 서영님께서 설정 해 놓으신 GTM id로 변경해야 할 것 같습니다~!! (기존에 comment 컴포넌트 id로 동일하게 가져가거나(id="commentLoginBtn"), 새롭게 설정(id="commentFormLoginBtn")하는 것 중에 고민해봐야 할 것 같아요!!
There was a problem hiding this comment.
헉 그렇군요 소현님..!!! 바로 고쳐놓도록 하겠습니당 감사해요!! 😋
Eugene-A-01
added a commit
that referenced
this pull request
Jun 5, 2024
…247) * Fix: 댓글 기능 추가(댓글 줄바꿈 표시, 비회원 댓글창 클릭시 로그인 모달, 답글 작성자 프로필 클릭 로직 추가), 인트로 페이지 영상 ios 자동재생 문제 테스트 (#242) * Feat: 답글 프로필 클릭시 해당 유저 마이리스트 페이지도 이동하도록 하는 기능 구현 * Fix: 비회원 댓글창 클릭시 로그인 모달 뜨게 구현 * Feat: textarea 출력시 줄바꿈 인식되게 수정 * Fix: 인트로 페이지 영상 ios에서 자동재생 안되는 문제 (테스트) * Fix: 답글도 줄바꿈 되도록 수정 * Fix: 댓글 작성 폼 모달 GTM id 수정 * Refactor: refreshToken 만료시간 재설정 (#241) * Style: 주석 정리 * Feat: useUser store에 사용하지 않는 accessToken 저장 로직 삭제 (사용자 id값만 저장되도록 수정) * Feat: refreshToken 쿠키 만료시간 재설정 * Feat: 개인정보 처리방침 페이지 연결 추가 * fix: 콜렉션 페이지 bottom nav에 가려지는 하단부 여백 추가 --------- Co-authored-by: 강나현 <[email protected]> Co-authored-by: Sohyun Park <[email protected]> Co-authored-by: Seoyoung <[email protected]>
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.
개요
작업 사항
관련 이슈 (optional)
closed: #194
스크린샷
댓글 줄바꿈
비회원 댓글창 클릭시
리뷰어에게