Skip to content

Conversation

@hongggyelim
Copy link
Collaborator

@hongggyelim hongggyelim commented Dec 11, 2024

작업 내용

  • edit페이지에서 쿼리 데이터로 초기값 설정
  • InputDropdown 컴포넌트에서 핸들러 수정 -> isDirty 활성화
  • css 일부 수정
  • 작성 취소 버튼 시 확인 모달 연동
    image

@hongggyelim hongggyelim added 📦️ Chore 자잘한 수정 🐞 Fix 버그 수정에 대한 커밋 labels Dec 11, 2024
@hongggyelim hongggyelim self-assigned this Dec 11, 2024
@github-actions
Copy link

github-actions bot commented Dec 11, 2024

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

@cccwon2 cccwon2 requested review from cccwon2 and yyezzzy December 11, 2024 13:29
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 Dec 11, 2024

Choose a reason for hiding this comment

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

image

제가 (albaform) 하위에 alba/[formid] 를 못보고 바깥 쪽에 alba/[formid] 만들어서 예림님 위치로 옮겼더니 (albaform) 하위 layout 때문에 css를 아무리 수정해도 잘 안되길래 못 합쳤어요 ㅠ 같은 폴더가 중복되버렸어요

Copy link
Collaborator Author

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.

아니면 alba 폴더 합치고 경로에 따라 레이아웃 분기 처리 하는게 나을까요??
(edit, addform, apply -> (albaform) 폴더 내의 레이아웃 적용
& 그 외 (= /alba만 있는 경우) 예지님 레이아웃 적용

@yyezzzy yyezzzy self-requested a review December 12, 2024 02:51
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.

수고하셨습니다 ~

@hongggyelim hongggyelim merged commit 0e87c7b into dev Dec 12, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📦️ Chore 자잘한 수정 🐞 Fix 버그 수정에 대한 커밋

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants