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

Fix: request -> axios로 변환 #23

Closed
kimseungki94 opened this issue Dec 23, 2022 · 0 comments · Fixed by #24
Closed

Fix: request -> axios로 변환 #23

kimseungki94 opened this issue Dec 23, 2022 · 0 comments · Fixed by #24
Assignees
Labels
🐞 bug & fix 예기치 않은 문제 또는 버그 발생, 수정 시 사용합니다.

Comments

@kimseungki94
Copy link
Collaborator

❗About

  • 요청을 두번 해야 데이터를 받는 이슈를 해결하기 위해 콜백기반 axios로 수정
  • request의 경우 promise가 아니라서, 많은 코드에 promise나 async/await 를 사용 시, 가독성이 다소 떨어질거라 판단하여, axios 사용
  • axios와 fetch 중 어떤걸 쓸지 고민이 많았지만, axios가 지정한 시간 초과 시, 별도의 작업을 처리하는 기능을 제공해주는 등 더 다양한 기능이 있다고 판단, axios로 수정을 하여, 해당 이슈를 해결할 계획

📝 To do

  • 유튜브, 사람인 크롤링 라우팅 JS 파일 수정
@kimseungki94 kimseungki94 added the 🐞 bug & fix 예기치 않은 문제 또는 버그 발생, 수정 시 사용합니다. label Dec 23, 2022
kimseungki94 added a commit that referenced this issue Dec 23, 2022
Feat : 요즘 IT, 유튜브, 사람인 Web API 모듈 수정(request -> axios)
@kimseungki94 kimseungki94 reopened this Dec 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug & fix 예기치 않은 문제 또는 버그 발생, 수정 시 사용합니다.
Projects
None yet
2 participants