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

[FE] 일정 표시 문구 개선 #905

Merged
merged 3 commits into from
Dec 27, 2023

Conversation

wzrabbit
Copy link
Collaborator

@wzrabbit wzrabbit commented Dec 23, 2023

[FE] 일정 표시 문구 개선

이슈번호

close #904

PR 내용

본 PR에서는 일정을 클릭할 경우 표시되는 문구를 좀 더 깔끔한 문구로써 보이도록 개선하였다. 구체적으로는 아래의 사항을 개선하였다.

  1. 2023-01-03 00:00 ~ 2023-01-03 23:59 와 같은 하루짜리 종일 일정은 2023-01-03 과 같이 변경하였다.
  2. 2023-01-03 00:00 ~ 2023-01-07 23:59 와 같은 여러 날짜리 종일 일정은 2023-01-03 ~ 2023-01-07 과 같이 변경하였다.
  3. 2023-01-03 03:00 ~ 2023-01-03 03:00 와 같은 시작 일정과 마감 일정이 완전히 같은 경우에는 2023-01-03 03:00 과 같이 변경하였다.
  4. 위의 어떠한 경우에도 속하지 않는 경우는 본래의 일정 정보를 그대로 보여준다.

3번 케이스의 경우 현재 PR 기준으로 같은 시간으로 시작해 같은 시간으로 끝나는 일정을 생성할 수 없도록 아직 막혀있으므로 msw 등의 모킹 데이터를 이용하여 테스트해 보아야 변경 사항이 보일 것이다. 유틸 함수의 테스트 코드를 구현한 것도 작동함을 보이기 위함이다.

참고 자료

image

의논할 사항

의논했던 요구사항에 기반하여 구현했지만 좀 더 개선할 수 있어보여서 조건을 더 만들고 문구도 더 추가해 보았어. 어떤 것 같아?

@wzrabbit wzrabbit self-assigned this Dec 23, 2023
@wzrabbit wzrabbit added this to the V2 1차 스프린트 milestone Dec 23, 2023
@wzrabbit wzrabbit marked this pull request as ready for review December 23, 2023 18:56
Copy link
Collaborator

@hafnium1923 hafnium1923 left a comment

Choose a reason for hiding this comment

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

구현하느라 고생했어! 이전PR과 어떤 관계가 있을진 잘 모르겠어서 어프루브만 해둘게! 문제없다면 바로 머지 해줘

@wzrabbit wzrabbit merged commit 7319d62 into develop Dec 27, 2023
1 check passed
@wzrabbit wzrabbit deleted the feat/fe/일정-표시-문구-개선 branch December 27, 2023 10:12
@wzrabbit
Copy link
Collaborator Author

두 PR은 서로 의존/충돌하지 않게끔 작업했어, 머지 완료!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 종일 일정에 한하여 일정 정보에서 표시되는 문구를 개선
2 participants