Skip to content

Comments

Feat/40/mypage refactor#126

Merged
cksrlcks merged 3 commits intomainfrom
feat/40/mypage-refactor
Feb 13, 2025
Merged

Feat/40/mypage refactor#126
cksrlcks merged 3 commits intomainfrom
feat/40/mypage-refactor

Conversation

@cksrlcks
Copy link
Collaborator

@cksrlcks cksrlcks commented Feb 13, 2025

❓이슈

✍️ Description

기존 user 쪽 schema를 건드린게 있어서, 도균님도 같이 리뷰 걸어두었습니다.

이미지 업로드 개선

  • 타입 및 스키마 수정
    • 응답 이미지 url이 nullable이라 null 들어올 수 있도록 개선
    • zod validation에 용량 추가
    • 타입중에 'URL' instance를 적은곳 삭제 (이건 타입이 안맞는것 같아요, 객체로 찍혀서요)
    • 이미지업로드 컴포넌트에 defaultValue 프롭스 추가 (추가로 올린 이미지를 지우면, 원래 이미지가 보이게)
    • 이미지업로드 컴포넌트에 null로 초기화도록 개선

내정보 페이지 개선

  • 내정보를 ssr로 불러오도록 개선
  • 폼내부 공용 컴포넌트로 교체
  • 폼내부 로직 개선
  • ssr데이터로 프로필 이미지를 default value로 받았을경우, 초기화하는 버튼 추가 (신규)
image

✅ Checklist

PR

  • Branch Convention 확인

    feat/* 기능 구현, fix/* 버그 수정, refactor/* 개선

  • Base Branch 확인
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test

  • 로컬 작동 확인

Additional Notes

  • (없음)

- 타입에 null 추가 (api에서 응답이 null이 올 수 있음)
- 쿼리 개선 (zod validation에 용량 추가)
- defaultValue 추가 (추가로 올린 이미지를 지우면, 원래 이미지가 보이게)
- 내정보를 ssr로 불러오도록 개선
- 폼내부 공용 컴포넌트로 교체
- 폼내부 로직 개선
@cksrlcks cksrlcks added Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링 labels Feb 13, 2025
@cksrlcks cksrlcks linked an issue Feb 13, 2025 that may be closed by this pull request
7 tasks
@vercel
Copy link

vercel bot commented Feb 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
taskify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 13, 2025 7:12am

Copy link
Owner

@ToKyun02 ToKyun02 left a comment

Choose a reason for hiding this comment

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

로직 변경 사항 확인했습니다!

나중에 스켈레톤 UI로 로딩 화면 보여주면 더 예뻐질 것 같네요 ㅎㅎ

@cksrlcks cksrlcks merged commit 85f391e into main Feb 13, 2025
3 checks passed
rak517 pushed a commit that referenced this pull request Feb 13, 2025
* ♻️ refactor : 이미지 업로드 개선
- 타입에 null 추가 (api에서 응답이 null이 올 수 있음)
- 쿼리 개선 (zod validation에 용량 추가)
- defaultValue 추가 (추가로 올린 이미지를 지우면, 원래 이미지가 보이게)

* ♻️ refactor : 내정보 페이지 개선
- 내정보를 ssr로 불러오도록 개선
- 폼내부 공용 컴포넌트로 교체
- 폼내부 로직 개선

* 🐛 fix : 헤더 내정보에 프로필 이미지 노출 추가
rak517 added a commit that referenced this pull request Feb 13, 2025
* ✨ feat: 할일 카드 모달 생성

* ✨ feat: 할 일 카드 상세정보 모달 구현

* 🐛 fix: 드롭다운, 이미지 수정

* 🔥 remove: 불필요한 console.log 삭제

* ♻️ refactor: confirm 사용 부분 useConfirm으로 교체

* Feat/40/mypage refactor (#126)

* ♻️ refactor : 이미지 업로드 개선
- 타입에 null 추가 (api에서 응답이 null이 올 수 있음)
- 쿼리 개선 (zod validation에 용량 추가)
- defaultValue 추가 (추가로 올린 이미지를 지우면, 원래 이미지가 보이게)

* ♻️ refactor : 내정보 페이지 개선
- 내정보를 ssr로 불러오도록 개선
- 폼내부 공용 컴포넌트로 교체
- 폼내부 로직 개선

* 🐛 fix : 헤더 내정보에 프로필 이미지 노출 추가

* 🐛 fix: useParams로 방문중인 대시보드 표시 개선 (#128)

Co-authored-by: chanki.kim <chanki.kim89@gmail.com>

* 💄 style: 로그인,회원가입 페이지 로고 png->svg변경 (#125)

* ♻️ refactor: 코드 리뷰 반영

* 🐛 fix: Avartor 컴포넌트 profileImageUrl 오류 해결

---------

Co-authored-by: 김찬기 (Chanki Kim) <44834561+cksrlcks@users.noreply.github.com>
Co-authored-by: SeokChan-Lee <melee0826@gmail.com>
Co-authored-by: chanki.kim <chanki.kim89@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Medium 🔖 우선순위 보통 Type: Refactor ♻️ 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: [페이지] 내 페이지 완성

2 participants