[feat] integrate user api for mypage #14

najitwo · 2024-11-17T00:53:15Z

📌 Related Issue

close #7

📝 Description

마이페이지 유저 API 연동
로그인한 유저 정보 조회 및 수정

…into 7-feat-integrate-user-api-for-mypage

najitwo · 2024-11-17T00:55:00Z

.eslintrc.json

  "rules": {
    "prettier/prettier": ["error"],
-    "no-unused-vars": "warn",
    "@typescript-eslint/no-unused-vars": "warn"


이거 둘이 같이 있으면 충돌나서 경고 밑줄이 이상하게 뜨더라고요
저희 어차피 typescript만 사용하니까 위에건 지웠습니다.

devmanta

👍 고생하셨습니다
보니까 각자 api호출하는 부분 구현해보는거라서 중복되는데
서로 어떻게 구현했는지 보고 보완해서 통합작업 진행해도 좋을것같습니다~

+아그리고 폴더구조로 router설정이 되다보니 라우팅 안하려고하는 애는 앞에 private느낌으로다가 _ 넣는다고 하더라구요..?
mypage하위에 보면 components 디렉토리 등등 있는데 라우팅 필요하지 않는애들은 _ 붙여서 사용하는건 어때요?

devmanta · 2024-11-17T04:51:54Z

src/app/mypage/components/FileInput.tsx

    }
  };

+  useEffect(() => {


use client 최상위에 선언 안해줘도 동작하나보군요?

부모컴포넌트가 'use client' 로 되어있으면 필요 없긴 한데...
공용으로 빼려면 고려해볼 사항이네요..!

devmanta · 2024-11-17T04:53:12Z

src/app/mypage/components/FileInput.tsx


-export default function FileInput({ name, setValue }: FileInputProps) {
-  const [preview, setPreview] = useState('');
+export default function FileInput({ name, setValue, url, id }: FileInputProps) {


요기서 name은 항상 'image' 문자열 고정값 맞을까요?
그럼 파라미터로 안받아도 되는..느낌일까요..?

네 image file input 용도로만 사용할 거 같아서 'image' 고정입니다.
파라미터는 없어도 되겠네요 👀

devmanta · 2024-11-17T05:02:32Z

src/app/mypage/page.tsx

+  const { login, getMe } = useAuth();
+
+  const handleLoad = async () => {
+    await login();


이거는 로그인 안되어있으면 강제로그인 시키는 건가요?
만약에 그런거라면
제가 작업하다가 로그인해야지 들어올수있는경로는
accessToken 없으면 login페이지로 redirect시키는 middleware추가해놨는데
요거 사용해봐도 좋을것같습니당 (다음PR에 올리려고 대기중..)

login 관련 로직들은 제가 그냥 테스트할때 편하게하려고 넣은겁니다..!
정상적인 로직이라면 로그인 후 접근이 가능해야겠죠..!

devmanta · 2024-11-17T05:03:27Z

src/app/mypage/store/authStore.ts

이부분은 승엽님 담당하는 부분인것 같은데 승엽님 어떻게 구상하고 계신지 상의해보고 공통적용해보시죠~!

devmanta · 2024-11-17T05:09:12Z

src/app/mypage/hooks/useAuth.ts

+    }
+  };
+
+  const login = async () => {


이건 개발 임시용인걸까요?!

devmanta · 2024-11-17T05:13:30Z

src/app/mypage/lib/authHelper.ts

여기서 행해지는 내용을보면 profile이랑 비밀번호 업데이트 하는 api호출을 하는것 같은데
파일명이 authHelper라서 약간 음... 좀 다른느낌?이 들긴합니다
auth라고 하면 인증/인가 관련된 무언가 일것같은데 보면은 사용자정보 업데이트 하는 내용이들어있어가지구..

파일명에대해 한번 더 생각해보겠습니다.. 어렵네요 🤔

devmanta · 2024-11-17T05:13:58Z

src/app/mypage/lib/authHelper.ts

+    if (error instanceof Error) {
+    }


요내용은 아직 안채워진거죠?ㅋㅋ

devmanta · 2024-11-17T05:17:36Z

src/app/mypage/store/authStore.ts

+  setUser: (user: User | null) => void;
+}
+
+const useAuthStore = create<AuthState>((set) => ({


오.. zustand 이렇게만 쓰면 촵촵 다되는건가요? 👍
근데 그럼 얘는 context처럼 적용범위를 children으로 �감싸지 않는데
적용범위는 항상 전체범위(?)에 적용되는건가요?

전체 범위는 아니고
적용 범위는 상태를 사용하는 컴포넌트만 해당합니다!

devmanta · 2024-11-17T05:18:45Z

src/app/mypage/types/user.ts

요것도 통합대상..!

* ✨ feat: integrate user API for mypage * ♻️ refactor: improve form component for readability * 🔥 comment: remove comment * 🔥 remove: remove unnecessary code * ♻️ refactor: update FileInput types for flexibility

najitwo added 3 commits November 17, 2024 03:10

✨ feat: integrate user API for mypage

bf1424e

Merge branch 'develop' of https://github.com/najitwo/10-Sprint-Taskify …

84291cf

…into 7-feat-integrate-user-api-for-mypage

♻️ refactor: improve form component for readability

8a1331e

najitwo added ✨Feature Request for a new feature or functionality 📬API Tasks related to integrating or connecting to external APIs labels Nov 17, 2024

najitwo self-assigned this Nov 17, 2024

najitwo linked an issue Nov 17, 2024 that may be closed by this pull request

[feat] integrate user API for mypage #7

Closed

3 tasks

najitwo commented Nov 17, 2024

View reviewed changes

🔥 comment: remove comment

82b0048

najitwo requested review from a user, devmanta and naseungyeop November 17, 2024 01:24

najitwo added 2 commits November 17, 2024 10:53

🔥 remove: remove unnecessary code

4b69065

♻️ refactor: update FileInput types for flexibility

46be792

devmanta approved these changes Nov 17, 2024

View reviewed changes

najitwo merged commit 186c0bb into develop Nov 17, 2024

najitwo deleted the 7-feat-integrate-user-api-for-mypage branch November 19, 2024 03:02

[feat] integrate user api for mypage #14

[feat] integrate user api for mypage #14

Uh oh!

Conversation

najitwo commented Nov 17, 2024

📌 Related Issue

📝 Description

Uh oh!

Choose a reason for hiding this comment

Uh oh!

devmanta left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants