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] FIX: 수지회 신청 날짜 형식 수정 #1718 #1719

Merged
merged 10 commits into from
Dec 26, 2024

Conversation

jihyunk03
Copy link
Collaborator

@jihyunk03 jihyunk03 commented Dec 26, 2024

해당 사항 (중복 선택)

  • FEAT : 새로운 기능 추가 및 개선
  • FIX : 기존 기능 수정 및 정상 동작을 위한 간단한 추가, 수정사항
  • BUG : 버그 수정
  • REFACTOR : 결과의 변경 없이 코드의 구조를 재조정
  • TEST : 테스트 코드 추가
  • DOCS : 코드가 아닌 문서를 수정한 경우
  • REMOVE : 파일을 삭제하는 작업만 수행
  • RENAME : 파일 또는 폴더명을 수정하거나 위치(경로)를 변경
  • ETC : 이외에 다른 경우 - 컨벤션 맞춰 수정

설명

[수요지식회]

  1. 수지회 신청 폼에서 신청 버튼을 누를 때, 신청한 날짜에서 연도는 신청 날짜 기준으로 보내지는 부분 수정
    • getFullYear() 사용 -> api에서 받아오는 값을 y/M/d 형식으로 바꾸어 출력시만 M/d로 변형하고, 연도도 해당 정보에서 가져오는 것으로 변경
  2. presentationLocation의 기본 값에 대해 수지회를 만들때 정해졌던 3층이 아닌 지하 1층으로 바뀜에 따라 수정
  3. 로컬에서 테스트하기 위한 케이스 5월까지 추가 (-> 현재를 기준으로 3개월만 보이는지 확인용)

[주의 해야 할 부분]

  1. 한달에 발표가 재단 사정 상 2번이 불가능 한 경우 -> 추가, 삭제 관련 내용이 필요함
  2. 현재 백엔드에서는 valid-date에 대해 현재 날(day) 기준, 3개월 이내의 더미에 대해 가능한 날짜에 대해 반환
    • 만약에, 현재 스케쥴러(1달에 1번 1달치)에서 변경될 경우(3개월 이후의 더미가 생성되어 있을 경우), 백엔드에서는 가능한 날짜로 넘어와서 신청이 되어도, 프론트 화면에서 볼 수 없음
    • 즉, 이 경우에는 valid-date의 기준을 '달(month)'을 기준으로 변경해야 함

#1718

@jihyunk03 jihyunk03 added FE Frontend tasks FIX Fix former works better labels Dec 26, 2024
@jihyunk03 jihyunk03 self-assigned this Dec 26, 2024
@jihyunk03 jihyunk03 linked an issue Dec 26, 2024 that may be closed by this pull request
Copy link
Collaborator

@wet6123 wet6123 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.

backend/database/cabi_local.sql Outdated Show resolved Hide resolved
@jihyunk03 jihyunk03 requested a review from wet6123 December 26, 2024 06:04
(13, NULL, '2025-04-09 14:00:00', 'dummy', 'BASEMENT', 'EXPECTED', 'HALF', 'dummy', 'dummy', NULL);
(14, NULL, '2025-04-23 14:00:00', 'dummy', 'BASEMENT', 'EXPECTED', 'HALF', 'dummy', 'dummy', NULL);
(15, NULL, '2025-05-07 14:00:00', 'dummy', 'BASEMENT', 'EXPECTED', 'HALF', 'dummy', 'dummy', NULL);
(16, NULL, '2025-05-21 14:00:00', 'dummy', 'BASEMENT', 'EXPECTED', 'HALF', 'dummy', 'dummy', NULL);
Copy link
Collaborator

Choose a reason for hiding this comment

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

바뀐 거 확인했습니다~~ 굳굳!!

@jihyunk03 jihyunk03 requested a review from Minkyu01 December 26, 2024 06:49
Copy link
Collaborator

@Minkyu01 Minkyu01 left a comment

Choose a reason for hiding this comment

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

저번에 못한 리뷰 지금 답니다.. 고친 부분 잘 작동되네요! 고생하셨습니다.

@jihyunk03 jihyunk03 merged commit c6afa15 into dev Dec 26, 2024
1 check passed
@jihyunk03 jihyunk03 deleted the fe/dev/fix-presentation-register#1718 branch December 26, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE Frontend tasks FIX Fix former works better
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] 수지회 신청 날짜(연도) 관련 이슈
4 participants