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

refactor: 카테고리, 일시 선택 순서 변경 및 리팩터링 (스타카토 생성/수정) #564

Open
6 tasks done
s6m1n opened this issue Dec 1, 2024 · 1 comment · May be fixed by #567
Open
6 tasks done
Assignees
Labels
refactor 리팩토링 (변수 및 메서드 네이밍 변경) ✅ test 테스트 (테스트 코드 추가, 수정, 삭제: 비즈니스 로직에 변경 없음)

Comments

@s6m1n
Copy link
Member

s6m1n commented Dec 1, 2024

🤮 As Is

공유하기 구현에 앞서 스타카토 생성/수정 화면카테고리, 일시 선택 기능을 리팩터링합니다.

🤩 To Be

  • 카테고리, 일시 선택 로직 테스트 추가
  • 카테고리, 일시 선택 순서 변경 (@linirini 제안)
  • 카테고리, 일시 선택 코드 리팩터링

용어 정리

  • 자동 선택 : 생성 페이지 진입 시 아래처럼 자동으로 선택되어있는 기본 값
image
  • 선택 가능 : 사용자가 선택 가능한 후보 목록
image

순서 변경 후 예상 로직

카테고리 특정 X (메인 화면에서 생성) 카테고리 특정 O (카테고리 조회 화면에서 생성)
image image
자동 선택된
일시
현재 일시 현재 일시와 가장 가까운 해당 카테고리 범위 내 일시
자동 선택된
카테고리
현재 날짜에서 선택할 수 있는 첫 번째 카테고리 해당하는 카테고리
선택 가능한
일시
현재 기준 ±10년에 해당하는 날짜, 모든 시간 제공 해당 카테고리 기간 내 날짜, 모든 시간 제공
선택 가능한
카테고리
현재 날짜에서 선택할 수 있는 모든 카테고리 해당하는 카테고리
카테고리
변경 가능
O X
일시 변경 시 날짜 기준 선택 가능 카테고리 후보 다시 로드 고정 카테고리이므로 고려할 필요 X

😇 이때까지 끝낼게요!

12월 8일

🙇‍♀️이슈 확인했어요:)

팀원에게 이슈 확인을 부탁해요!

@s6m1n s6m1n added refactor 리팩토링 (변수 및 메서드 네이밍 변경) ✅ test 테스트 (테스트 코드 추가, 수정, 삭제: 비즈니스 로직에 변경 없음) labels Dec 1, 2024
@s6m1n s6m1n self-assigned this Dec 1, 2024
@linirini
Copy link
Contributor

linirini commented Dec 2, 2024

플로우에 따라 로직을 다르게 가져가는 것이 복잡하지 않다면 이대로 가도 될 것 같아요!
(만약, 복잡할 것 같으면 홈화면에서 스타카토 추가 로직을 따라가도 괜찮을 것 같습니다!)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩토링 (변수 및 메서드 네이밍 변경) ✅ test 테스트 (테스트 코드 추가, 수정, 삭제: 비즈니스 로직에 변경 없음)
Projects
Status: No status
2 participants