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.

Sign up for GitHub

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

Jump to bottom

feat-fe: 공고 대시보드 제목 영역에 모집일자 및 상태 정보 추가 #802

Merged
merged 11 commits into from
Oct 15, 2024
Merged

feat-fe: 공고 대시보드 제목 영역에 모집일자 및 상태 정보 추가 #802

merged 11 commits into from
Oct 15, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 11, 2024 •
edited by seongjinme
Loading

Original issue description

목적

  • 공고 대시보드 제목 영역에 모집일자 및 모집 여부에 대한 정보를 추가합니다.

작업 세부사항

  • 공고 대시보드 제목 하단에 모집일자, 모집상태 추가

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!

DA_HEADER_01

목적

  • 공고 대시보드 제목 영역에 모집일자 및 모집 여부에 대한 정보를 추가합니다.

작업 세부사항

  • 공고 대시보드 제목 하단에 모집일자, 모집상태 추가
  • "공고로 이동" UI 요소를 DashboardHeader 컴포넌트와 병합
  • 변경된 Process 목록 조회 API 스펙에 맞춰 관련 Hook 및 Type 코드 업데이트

참고 이미지

개선 전 화면
image
개선 후 화면
image

관련 이슈

@github-actions github-actions bot added feature 새로운 기능 frontend 프론트엔드 labels Oct 11, 2024
@github-actions github-actions bot added this to the 스프린트 6.0 milestone Oct 11, 2024
@seongjinme seongjinme marked this pull request as ready for review October 15, 2024 06:14
@github-actions GitHub Actions
Copy link
Contributor Author

1728972857.020319

@github-actions GitHub Actions
Copy link
Contributor Author

1728972867.118179

lurgi
lurgi requested changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@lurgi lurgi left a comment •
edited
Loading

Choose a reason for hiding this comment

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

UI확인 하였습니다! 고생하셨어요~
코드단에서도 딱히 피드백 드릴 부분 없어보여용~
다만
image
모집 완료가 기대되지만 모집 진행중으로 나타나요.
시간관련 버그가 있어보입니다! 스토리북에서만의 문제인진 잘 모르겠어요. 확인 부탁드려요!

@seongjinme
Copy link
Contributor

UI확인 하였습니다! 고생하셨어요~ 코드단에서도 딱히 피드백 드릴 부분 없어보여용~ 다만 image 모집 완료가 기대되지만 모집 진행중으로 나타나요. 시간관련 버그가 있어보입니다! 스토리북에서만의 문제인진 잘 모르겠어요. 확인 부탁드려요!

시간 체크는 getTimeStatus 유틸 함수를 그대로 사용하고 있어요. 현재 우리는 시각(시/분/초) 지정 없이 날짜 단위로만 모집 기간 여부를 검증하기 때문에 이렇게 나온 것 같아요. 그리고 아쉽게도, Storybook 공식 문서를 보니 날짜만 선택 가능한 Datepicker는 따로 지원되지 않네요 😓

llqqssttyy
llqqssttyy approved these changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@llqqssttyy llqqssttyy left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

frontend/src/components/recruitment/RecruitmentCard/index.tsx
Comment on lines +50 to +53
<RecruitmentStatusFlag
startDate={startDate}
endDate={endDate}
/>
Copy link
Contributor

Choose a reason for hiding this comment

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

깔끔해졌네용👍

@seongjinme
Copy link
Contributor

seongjinme commented Oct 15, 2024 •
edited
Loading

렛서(@llqqssttyy )의 아이디어에 따라 대시보드 제목 영역의 "공고로 이동" 버튼 요소가 아래와 같이 조정되었습니다.

image

관련하여 렛서의 추가 커밋 내용을 아래와 같이 공유합니다. 이 변경 내용대로 본 PR을 Merge합니다.

@seongjinme seongjinme merged commit 590cdc8 into fe/develop Oct 15, 2024
10 of 11 checks passed
@seongjinme seongjinme deleted the fe-801-DA_HEADER_01 branch October 15, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@llqqssttyy llqqssttyy llqqssttyy approved these changes

@lurgi lurgi lurgi approved these changes

Assignees

@seongjinme seongjinme

Labels
feature 새로운 기능 frontend 프론트엔드
Projects
2024-cruru
Status: 완료
Milestone
스프린트 6.0
Development

Successfully merging this pull request may close these issues.
3 participants
@seongjinme @llqqssttyy @lurgi