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

Feat: 내 체험 관리, 체험 등록 form #35

Merged
merged 14 commits into from
Jul 16, 2024
Merged

Conversation

eunji-0623
Copy link
Owner

@eunji-0623 eunji-0623 commented Jul 16, 2024

🔎 작업 내용

  • form 관련 변경 내역
    • 주소 input Daum 라이브러리 사용하도록 변경
    • 제출 시 내 체험 목록 페이지로 이동하도록 변경
  • 내 체험 목록 페이지 변경 내역
    • api 연결 (목록 불러오기, 삭제)
    • 무한 스크롤 적용
    • 사이드 네비게이션 연결

⚠️ 이슈

  • 반응형을 고려하지 않고 작업해서 추후 반응형 디자인 수정 필요
  • 내 체험 목록 페이지에서 삭제 모달이 뜰 때 화면이 살짝 움직이는 현상이 발생
  • 내 체험 목록 페이지 로딩 처리 추가 필요
  • Daum 주소 입력 input 디자인 수정 필요

📢 전달사항

  • Daum 우편 관련 라이브러리를 추가했습니다. : react-daum-postcode

image
image

@eunji-0623 eunji-0623 added the 라이브러리 추가 추가한 라이브러리가 있습니다! label Jul 16, 2024
@eunji-0623 eunji-0623 requested review from MEGUMMY1 and Byukchong July 16, 2024 02:37
@eunji-0623 eunji-0623 self-assigned this Jul 16, 2024
@eunji-0623 eunji-0623 changed the title Feature 황은지 Feat: 내 체험 관리, 체험 등록 form Jul 16, 2024
Copy link
Collaborator

@MEGUMMY1 MEGUMMY1 left a comment

Choose a reason for hiding this comment

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

PR 본문에 추가된 라이브러리 이름이나 코드 추가 부탁드릴게요 🤗

Comment on lines +30 to +35
<DaumPostcode
style={style}
autoClose
onComplete={handleComplete}
onClose={handleClose}
/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

👩‍💻👍

Comment on lines 32 to 33
case '/myActivity':
setActiveButton('myActivity');
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 부분 URL 전체 소문자나 하이픈으로 수정하는 게 좋을 것 같습니다!!
https://devuna.tistory.com/79

Copy link
Collaborator

@Byukchong Byukchong left a comment

Choose a reason for hiding this comment

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

작업량 많은데 고생하셨습니다!

@eunji-0623 eunji-0623 merged commit 43b7050 into master Jul 16, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
라이브러리 추가 추가한 라이브러리가 있습니다!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants