Skip to content

hotfix: 기술스택 모달 생성 제한#381

Merged
davidpro08 merged 5 commits intomainfrom
fix/modal-count-limit
Feb 5, 2026
Merged

hotfix: 기술스택 모달 생성 제한#381
davidpro08 merged 5 commits intomainfrom
fix/modal-count-limit

Conversation

@as-zini
Copy link
Collaborator

@as-zini as-zini commented Feb 5, 2026

✅ 작업 내용

  • 전역적으로 모달이 열림 여부를 관리할 스토어 생성
  • 캔버스에 모달이 열려있으면 추가하기 버튼 클릭해도 모달 열리지 않도록 수정

📸 스크린샷 / 데모 (옵션)

2026-02-05.11.23.27.mov

💬 참고 사항

기술스택 모달을 두개 이상 생성하면 드래그앤드롭시 모달 속 아이템 요소가 사라지는 현상이 발생합니다.

2026-02-05.11.02.37.mov

아직은 원인을 찾지 못해서 급하게 모달 열림 여부를 전역에서 관리하고, 추가하기 버튼을 눌렀을 경우에 모달이 열려있는지 여부를 확인하여 열려있다면 아무동작도 일어나지 않도록 하는 방식으로 모달을 하나만 띄울 수 있게 막아두었습니다

정확한 원인을 찾거나, 더 좋은 방법으로 해결이 가능하다면 코멘트 남겨주시거나 PR 올려주세요!(불가능하다면 우선은 이 PR 반영해서 막아놓으면 될 것 같습니다...!)

@vercel
Copy link

vercel bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
web15-ipconfig-frontend Ready Ready Preview, Comment Feb 5, 2026 9:09pm

@auto-assign auto-assign bot requested review from Grit03, SnailW and davidpro08 February 5, 2026 14:30
@github-actions github-actions bot added the 🐛 fix 버그 수정 label Feb 5, 2026
Copy link
Collaborator

@SnailW SnailW left a comment

Choose a reason for hiding this comment

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

일단은 이 방법으로 막아두고 구조적으로 나은 방향은 차차 생각해보는게 좋을 것 같네요...
고생하셨습니다

Copy link
Member

@Grit03 Grit03 left a comment

Choose a reason for hiding this comment

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

네 일단 이 방법으로 해결해보고, 원인 저도 찾아볼게요!
빠르게 해결하느라 고생많으셧습니다!

Copy link
Collaborator

@davidpro08 davidpro08 left a comment

Choose a reason for hiding this comment

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

문제 상황 발견하고 해결하시느라 고생하셨습니다!

@davidpro08 davidpro08 merged commit 1a24350 into main Feb 5, 2026
5 checks passed
@davidpro08 davidpro08 deleted the fix/modal-count-limit branch February 5, 2026 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants