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/#198] 오전 저녁 선택시 오후 시간대 선택 막기 #204

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

simeunseo
Copy link
Member

@simeunseo simeunseo commented Oct 16, 2023

#198

todo

  • 오후 시간대가 포함될 시 alert 띄우기

📌 공유하고 싶은 부분

  • 시작시간과 종료시간의 선후가 올바른지 체크하는 if문 로직안에서, 만약 이 시간표가 오전/저녁 시간표일시 1시작 시간이 11:30 이하이면서 종료 시간이 18:00이후`인지 체크하는 로직을 추가하여 아니라면 alert를 띄우는 방법으로 해결했습니다.
  • 만약 오전 시간대(ex. 09:00-10:00)를 선택했다가 이를 오후 시간대(ex.18:00-19:00)로 바꾸고 싶으면, 시작 시간대를 저녁으로 옮기면 시작시간이 종료시간보다 늦어서 선택이 안되고, 종료 시간을 저녁으로 옮기면 선택하면 안되는 오후 시간대가 선택이되어 또 막히게 됩니다.... 따라서 이 부분에서는 그냥 사용자가 X표를 눌러서 다시 선택을 해야하는데... 흠... 재훈오빠랑 잠시 얘기해본 바로는 그냥 이게 최선일 것 같다는 생각입니다. 동헌 의견도 궁금합니다

📌 질문할 부분

  • "오후 12시부터 18시는 선택지에 포함될 수 없습니다." 카피 괜찮나요?

📌스크린샷

image

@simeunseo simeunseo added fix fix 은서 은서의 개발 라벨 labels Oct 16, 2023
@simeunseo simeunseo self-assigned this Oct 16, 2023
@vercel
Copy link

vercel bot commented Oct 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
asap ✅ Ready (Inspect) Visit Preview Oct 16, 2023 7:17am

@ljh0608
Copy link
Member

ljh0608 commented Oct 16, 2023

LGTM

@ljh0608 ljh0608 merged commit f4ba5ff into develop Nov 3, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix fix size/M 은서 은서의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants