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

fix: 스토리 우선순위 변경 로직 수정 #323

Merged
merged 5 commits into from
Aug 6, 2024

Conversation

surinkwon
Copy link
Collaborator

🎟️ 태스크

프로젝트 회원은 백로그에서 스토리 우선순위를 변경할 수 있다.

✅ 작업 내용

  • 스토리 우선순위 변경 로직 수정
  • 스토리별 페이지에서 완료된 스토리 제거
  • 에픽 색상 랜덤 지정 로직 수정
  • 에픽 드롭다운 길이 설정

🖊️ 구체적인 작업

스토리 우선순위 변경 로직 수정

  • 이전 로직은 index를 기반으로 우선순위를 변경할 스토리를 식별해서 여러 사람이 동시에 다른 스토리의 우선순위를 변경하면 의도하지 않은 스토리가 변경되는 문제가 발생했습니다. 이를 방지하기 위해 스토리의 아이디를 기반으로 식별하는 로직으로 변경했습니다.

에픽 색상 랜덤 지정 로직 수정

  • useMemo를 사용해 에픽 색상을 생성하고 있었어서 하나의 에픽을 생성하고 난후 에픽 드롭다운이 닫히지 않으면 계속 똑같은 색상으로 에픽이 생성되었습니다. 이를 상태로 관리하도록 해서 새로운 에픽 생성 시 다시 랜덤한 색상이 배정되도록 했습니다.

@surinkwon surinkwon merged commit b693606 into dev Aug 6, 2024
1 check passed
@surinkwon surinkwon deleted the feature/drag-and-drop branch August 6, 2024 12:51
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.

1 participant