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

[Others] PartyAdmin추가에 따른 Axios instance 리팩토링 #1295

Open
izone00 opened this issue Mar 13, 2024 · 2 comments
Open

[Others] PartyAdmin추가에 따른 Axios instance 리팩토링 #1295

izone00 opened this issue Mar 13, 2024 · 2 comments
Labels
admin 관리자 페이지에서 발생한 issue refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue

Comments

@izone00
Copy link
Contributor

izone00 commented Mar 13, 2024

Desc

Axios instance가 party/admin 이라는 url이 새로 추가 되어 아래와 같이 작성되어있습니다.

Detail

Todo

Etc

Screen Shot 2024-03-13 at 4 15 41 PM
@izone00 izone00 added admin 관리자 페이지에서 발생한 issue refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue labels Mar 13, 2024
@nerdchanii
Copy link

#1579 에서 사용하고 있는 axiosInstance들의 onFulfilled, onError에 해당하는 콜백을 합쳐서 분리했습니다. 이과정에서 onError 콜백이 도달하지 않는 콜백으로 확인했습니다. 기존 onError함수가 error를 잡았다가 던지는 행위가 전부라 필요한 이유가 조금 애매해 보이는데 이전 기수들에서는 onError핸들러에 에러가 도달했었는지 알 수 있을까요?:)

@nerdchanii
Copy link

#1579 이슈가 해결되는대로 함께 해당이슈를 닫을 예정입니다. 추후 추가적인 수정이 필요하다고 여겨지시면 이슈를 다시생성하거나 reopen 해주세요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin 관리자 페이지에서 발생한 issue refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue
Projects
None yet
Development

No branches or pull requests

2 participants