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

Y.Doc의 node에 isHolding property 추가 #208

Merged
merged 2 commits into from
Nov 19, 2024
Merged

Conversation

djk01281
Copy link
Collaborator

@djk01281 djk01281 commented Nov 19, 2024

🔖 연관된 이슈

📂 작업 내용

  • Y.Doc의 nodeisHolding property 추가.
  • dragStop 시, false로 설정해서 보내기.

📑 참고 자료 & 스크린샷 (선택)

스크린샷 2024-11-19 오후 6 20 35

📢 리뷰 요구사항 (선택)

  • isHolding입니다. 어떻게 고쳐도 마지막에 false로 2번을 보내네요.. 사실 디바운싱이 목적이었으니까 일단은 이것으로 충분하지 않을까 싶긴 합니다..
  • Object.entries()filter()를 사용한 부분이 있는데, eslint rule과 싸워서 빌드 성공하기 위한 엄청난 사투가 있었습니다.. 😇 원래 원했던 것 : const initialNodes = yNodes.map(({ isHolding, ...node }) => node);

@djk01281 djk01281 requested a review from yewonJin November 19, 2024 09:50
@djk01281 djk01281 added 🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우 labels Nov 19, 2024
@djk01281 djk01281 changed the title .Doc의 node에 isHolding property 추가 Y.Doc의 node에 isHolding property 추가 Nov 19, 2024
Copy link
Collaborator

@yewonJin yewonJin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

@yewonJin yewonJin merged commit 0772c25 into develop Nov 19, 2024
18 checks passed
@Tolerblanc Tolerblanc deleted the feature-fe-#207 branch November 20, 2024 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐰🐣 FE 프론트엔드 관련 이슈/PR 추적 라벨 ✨ Feature 기능 추가 및 구현, 제안과 관련된 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Y.Doc의 node에 isHolding property 추가
2 participants