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

사서 로테이션 관련 업데이트 #114

Merged
merged 4 commits into from
Jun 27, 2023
Merged

사서 로테이션 관련 업데이트 #114

merged 4 commits into from
Jun 27, 2023

Conversation

jhj9109
Copy link
Collaborator

@jhj9109 jhj9109 commented Jun 27, 2023

기능 개선 사항

  1. 이슈 108 로테이션 신청기간 조절
  • 기존 ISO 기준 4주차 월~일요일, 말일이 일요일이면, 로테이션이 확정되는 시간이 다음달 1일 00:00 인 문제 발생
  • 업데이트를 통해 ISO 기준 4주차 월~금요일로 신청 기간 변경, 최소한 주말간 로테이션을 확인하고, 필요하면 사서간 수정을 할 수 있는 기간을 부여하는 것을 목적
  1. 이슈113_2 사서 달력 페이지에서 비로그인 상태에서 클릭 버튼 동작 수정
  • 비로그인 유저가 달력 클릭시, 실제 요청으로 이어지기전 잡아내지 못했음
  • 해당 조건문 수정을 통해서 비로그인 유저가 해당 요청까지 도달하지 못하도록 수정
  1. 이슈 113_1 사서 신청하기 페이지에서 신청 or 신청 취소 작업 완료후 새로고침토록 수정
  • 기존에는 작업 완료 시에도 어떠한 UI 변화도 없어, 작업 상태에 대한 의문점이 들게 되는 상황
  • 가장 간단하게 새로고침을 통해서 UI변화를 부여하여 나의 작업 상태에 대해 인지할수있도록 수정
  • 현재는 window.location.reload() 로서 새로고침하나, 이는 리소스를 모두 다운로드해야하는 코드
  • 추후 개선을 통해서, UI적으로만 변화가 일어나도록 관련 기능을 개선한 예정

close #108 #113

jhj9109 added 4 commits June 27, 2023 20:23
기존 코드에서는 title === null 비교를 활용중이였음.
auth가 없으면 title이 undefined되기 때문에, 통과되었음
title의 불린판정을 활용하여도 사이드이펙트는 없을것으로 판단되어 수정
사서 로테이션 신청 기간을 ISO기준 4주차 월요일 ~ 금요일로 수정
- 기존에는 월요일 ~ 일요일
- 말일이 일요일인 경우, 로테이션이 확정이 다음달 1일 0시에 되는 문제 발생
- 최소한 주말에 확정된 일정을 확인할 수 있도록, 신청 기간을 금요일까지로 단축
로테이션 신청페이지에서 신청 완료 후, 혹은 취소 완료 후
어떠한 UI변화도 없어서 보완할 계획이였음.
새로고침의 역할을 하는 코드로 우선 구현하였으나,
추후 테스트 가능한 환경에서 코드 개발 후, 필요한 것만 갈아끼우는 방식의 개선된 새로고침 업데이트 예정
현재로선 isLoading이라는 state가 불필요하나, 이것은 추후 개선된 새로고침을 위함
@jhj9109 jhj9109 merged commit 9cbba39 into develop Jun 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[기능 개선 사항] 사서 로테이션 신청 기간에 대한 개선이 필요
1 participant