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

[REFACTOR] Refresh token 도입, Locker API 검증 #80

Merged
merged 13 commits into from
Feb 29, 2024

Conversation

selfishAltruism
Copy link
Contributor

@selfishAltruism selfishAltruism commented Feb 27, 2024

🚩 관련 이슈
resolve #79, #81

📋 PR Checklist

  • 로그인 유지 시스템 refresh token 도입
  • 일반 API access token 유효성 확인 및 재발급 구조 도입

📌 유의사항
✅ 테스트 결과
image

@selfishAltruism selfishAltruism added fix 올바르지 않은 동작, 버그 수정 feat 새로운 기능, 코드 추가 refactor 코드 리팩토링 labels Feb 27, 2024
@selfishAltruism selfishAltruism self-assigned this Feb 27, 2024
Copy link

netlify bot commented Feb 27, 2024

Deploy Preview for causw-develop ready!

Name Link
🔨 Latest commit 2200ef4
🔍 Latest deploy log https://app.netlify.com/sites/causw-develop/deploys/65e0663e332ab600087bafaa
😎 Deploy Preview https://deploy-preview-80--causw-develop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Feb 27, 2024

Deploy Preview for caucse ready!

Name Link
🔨 Latest commit 2200ef4
🔍 Latest deploy log https://app.netlify.com/sites/caucse/deploys/65e0663eea1d060008c7af9d
😎 Deploy Preview https://deploy-preview-80--caucse.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 87
Accessibility: 96
Best Practices: 92
SEO: 92
PWA: 80
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@selfishAltruism
Copy link
Contributor Author

지금 Back 서버에서 에러가 있어서 실 서버랑 문제 없는지 확인을 못한 상태인데, approve 해주시면 제가 확인한 다음에 merge 하겠습니다.

Copy link
Contributor

@rbgksqkr rbgksqkr left a comment

Choose a reason for hiding this comment

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

넵 고생하셨습니다!!! 재발급 확인 후 머지해주세요

@selfishAltruism selfishAltruism changed the title [REFACTOR] Refresh token 도입 [REFACTOR] Refresh token 도입, Locker API 검증 Feb 28, 2024
@selfishAltruism
Copy link
Contributor Author

Locker API 검증 commit도 추가되었습니다.

@selfishAltruism selfishAltruism merged commit 41b0c24 into develop Feb 29, 2024
@selfishAltruism selfishAltruism deleted the refactor/#79 branch March 11, 2024 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능, 코드 추가 fix 올바르지 않은 동작, 버그 수정 refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants