Skip to content

Conversation

@hongggyelim
Copy link
Collaborator

@hongggyelim hongggyelim commented Dec 11, 2024

주요 수정 사항

  • alba/[formId]/edit 페이지 추가 했습니다
  • 초기 데이터로 쿼리 데이터를 가져오도록 했습니다
    -> text input/area에만 반영되는 문제가 있습니다
    (addform페이지에서부터 연결되는 문제 인 것 같습니다 - 이어서 개선 작업 하겠습니다)
  • 비회원도 apply 할수있도록 route 수정했습니다
  • 알바폼 수정/삭제 (사장님) #24

기타 공유 사항

  • FormDetailResponse 타입에서 날짜 Date -> string으로 수정해습니다.
  • 이력서 업로드 안되는 문제 해결 (처음에 안되는것 확인했는데 디버깅 코드만 추가했는데 또 되네요? 확인 한번 해주세요~)
  • isEditing 로직을 분리했습니다
  • formatMoney 로직을 분리했습니다 (/util)
  • 폼 제출 시 이동 경로를 수정했습니다 (세부 페이지로) - 페이지 경로가 수정된다면 같이 수정해주세요

@cccwon2 cccwon2 requested review from cccwon2 and yyezzzy and removed request for cccwon2 December 11, 2024 05:47
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.

수고하셨습니다.

@github-actions
Copy link

github-actions bot commented Dec 11, 2024

💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-jobvbapcmy.chromatic.com/ # Chromatic에서 생성된 Storybook URL
🕖 Update: 2024년 12월 11일 16시 52분 28초 # 현재 시간

@hongggyelim hongggyelim added the 📬 API API 통신 관련 label Dec 11, 2024
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.

수고하셨습니다 ~

defaultValues: {
isPublic: false,
hourlyWage: 0,
hourlyWage: 10030,
Copy link
Collaborator

Choose a reason for hiding this comment

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

요게 어떤 값인가용

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

최저 시급 입니당 ㅎㅎ

@yyezzzy yyezzzy merged commit 7612a98 into dev Dec 11, 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 통신 관련

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants