Skip to content
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/#307] lineComment react-md-editor 적용 가능 한지 테스트 #308

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

krokerdile
Copy link
Collaborator

🎫 연관 티켓

#307

🙏 작업

  • 기존에 작업해둔 Line Comment #270 라인코멘트 훅이 정상적으로 react-md-editor에 적용되어지는 지 테스트

💁‍♂️ 어떻게?

  • Line Comment #270 에서 구성한 LineComment 내부의 Hook과 popover 컴포넌트 활용해서 진행

🙈 PR 참고 사항

문제점

기존에 Id를 기준으로 내용을 읽어와서 md editor 의 요소의 경우에는 Id가 따로 없다는 문제가 있음.
이를 해결 하기 위해서 따로 처리를 해주던가 해결할 방법을 찾아봐야 될 것 같음.

📸 스크린샷

스크린샷 2024-02-14 오후 3 56 58

🤖 테스트 체크리스트

  • 체크 미완료
  • 체크 완료

Related to 라인 코멘트, react md editor에 되는지 테스트 #307
@krokerdile krokerdile self-assigned this Feb 14, 2024
@Klomachenko
Copy link
Collaborator

image

@krokerdile
Copy link
Collaborator Author

감사하옵나이다

@krokerdile krokerdile merged commit 78f3a43 into fe/dev Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants