Skip to content

Conversation

@hongggyelim
Copy link
Collaborator

@hongggyelim hongggyelim commented Dec 9, 2024

알바폼 생성 페이지 (/addform)

추가 수정 필요한 부분

  • 일부 이전 선택 데이터 렌더 안됨 (선택한 날짜 / 이미지)

  • 브랜치가 꼬여서 새로운 브랜치로 PR올립니다

  • 아직 build 에러가 해결이 안됐어요 ㅠ

@github-actions
Copy link

github-actions bot commented Dec 9, 2024

💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-dvuqgxqimj.chromatic.com/ # Chromatic에서 생성된 Storybook URL
🕖 Update: 2024년 12월 09일 22시 50분 47초 # 현재 시간

@hongggyelim hongggyelim self-assigned this Dec 9, 2024
@hongggyelim hongggyelim added ⚙ Build 빌드 관련 파일 수정 📬 API API 통신 관련 labels Dec 9, 2024
@cccwon2 cccwon2 requested review from cccwon2 and yyezzzy and removed request for yyezzzy December 9, 2024 13:32
Copy link
Contributor

@cccwon2 cccwon2 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다.

Copy link
Collaborator

@yyezzzy yyezzzy left a comment

Choose a reason for hiding this comment

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

input이 많은데 수고하셨습니다 ~! 주석도 꼼꼼히 달아주셨네요 ㅎㅎ

Comment on lines +81 to +84
// 폼 제출 리액트쿼리
const mutation = useMutation({
mutationFn: async () => {
const excludedKeys = ["displayDate", "workDateRange", "recruitDateRange", "imageFiles"];
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

import InputDropdown from "@/app/components/button/dropdown/InputDropdown";

// 알바폼 만들기 - 사장님- 2-모집조건
export default function RecruitCondition() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

저희 페이지는 RecruitConditionPage 로 뒤에 Page 붙이는게 좋아보여요
회의 때 얘기해봐요 !

@cccwon2 cccwon2 merged commit 348eb60 into dev Dec 9, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📬 API API 통신 관련 ⚙ Build 빌드 관련 파일 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants