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

프론트의 auth기능 refactor #128

Open
2 tasks
seo-wo opened this issue Dec 15, 2023 · 1 comment · Fixed by #131
Open
2 tasks

프론트의 auth기능 refactor #128

seo-wo opened this issue Dec 15, 2023 · 1 comment · Fixed by #131
Assignees
Labels
refactor 코드를 리팩토링 할 때 사용합니다

Comments

@seo-wo
Copy link
Member

seo-wo commented Dec 15, 2023

리팩토링 된 백엔드 로그인 명세에 맞춰 프론트단에서도 수정이 있습니다.

  • signup 페이지 수정
  • 해당 페이지를 기존의 기능과 유사하게끔 scss를 포함하여 수정 예정입니다.
  • auth stroage
  • 로그인한 유저의 정보를 불러 오는 local strage에서 가져오는 auth stroage를 global state로 수정해볼 예정입니다.

이 외에도 추가될 내용들은 커멘트에 실시간으로 달아 공유 할 예정입니다.

@seo-wo seo-wo added the refactor 코드를 리팩토링 할 때 사용합니다 label Dec 15, 2023
@seo-wo seo-wo self-assigned this Dec 15, 2023
@seo-wo
Copy link
Member Author

seo-wo commented Dec 15, 2023

  • axios를 직접 사용하는 방식에서 axios 인스턴스를 만들어 사용하는 방식으로 바꿀 예정입니다.

seo-wo added a commit that referenced this issue Dec 18, 2023
use global state via recoil, when login and signup

#128
seo-wo added a commit that referenced this issue Dec 18, 2023
wip: need to refactor

#128
seo-wo added a commit that referenced this issue Dec 18, 2023
use api to post logout

#128
@seo-wo seo-wo linked a pull request Dec 18, 2023 that will close this issue
seo-wo added a commit that referenced this issue Dec 18, 2023
seo-wo added a commit that referenced this issue Dec 18, 2023
see detail src/services/apiClient.ts

#128
seo-wo added a commit that referenced this issue Dec 18, 2023
when login is needed, navigator client to home page

#128
@seo-wo seo-wo reopened this Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 코드를 리팩토링 할 때 사용합니다
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant