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

[패키지 업데이트] 패키지 업데이트 및 Axios버전업 #574

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

daepan
Copy link
Contributor

@daepan daepan commented Nov 12, 2024

  • Close #ISSUE_NUMBER

What is this PR? 🔍

  • 기능 :
  • issue : #

Changes 📝

  1. 전체적인 package 기능을 업데이트 했습니다.
  • @types dev 쪽으로 수정
  • 미사용 라이브러리 제거
  1. Axios 버전업을 진행하였습니다.

자세한 내용은 해당 링크를 참고해주세요!

ScreenShot 📷

Test CheckList ✅

  • test 1
  • test 2
  • test 3

Precaution

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

Copy link
Contributor

@MinGu-Jeong MinGu-Jeong left a comment

Choose a reason for hiding this comment

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

스테이지에 올리고 잘 동작하는지 확인해봅시다
수고하셨습니다!

"@types/react-query": "^1.2.9",
"@types/react-window": "^1.8.5",
"axios": "^0.27.2",
"axios": "^1.7.5",
Copy link
Contributor

Choose a reason for hiding this comment

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

오호.. 그래도 메이저 업데이트니깐 여기저기 잘 동작하는지 확인을 해봐야겠군여
특별히 기능적으로 달라진 부분이 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

axios 1버전의 큰 차이는 TypeScript 지원 강화, 보안 강화, ES 모듈 지원 등으로 현대적인 자바스크립트와 타입스크립트 프로젝트에서 더 편리하고 안전하게 사용할 수 있도록 개선된 버전이라고 할 수 있습니다.
기능적으로 크게 변화된 건 없고 TypeScript 지원에 대해서는 흥미로운 정보가 있어서 나중에 TypeScript 버전 업 트러블 슈팅을 하며 해당 apiClient 로직을 대대적으로 손봐야할 수 도 있겠네요
0.x 에서 버전

async function fetchUser(userId: number): Promise<AxiosResponse<User>> {
  return await axios.get<User>(`/api/users/${userId}`);
}

1.x에서의 TypeScript 사용

import axios from 'axios';

interface User {
  id: number;
  name: string;
}

// 명시적 타입 지정 없이도 `User` 타입을 추론
async function fetchUser(userId: number): Promise<User> {
  const response = await axios.get<User>(`/api/users/${userId}`);
  return response.data; // 자동으로 User 타입으로 추론됨
}

Copy link
Contributor

@chaeseungyun chaeseungyun left a comment

Choose a reason for hiding this comment

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

고고

@daepan
Copy link
Contributor Author

daepan commented Nov 21, 2024

요고 이번 배포 지나고 붙힐게요!!

Copy link
Contributor

@D0Dam D0Dam left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants