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

recoilの適用とGoogleソーシャルログイン機能の追加 #106

Merged
merged 5 commits into from
Apr 20, 2024

Conversation

dorimu0
Copy link
Contributor

@dorimu0 dorimu0 commented Apr 19, 2024

🔍 このPRで解決したい問題は何ですか?

  • recoilによるログイン状態管理追加及びGoogleソーシャルログイン機能を追加しました。

✨ このPRで主に変わったことは何ですか?

  • Recoilライブラリを通じたログイン関連ステータス管理を追加しました。それを基にGoogleソーシャルログイン機能を実装し、ユーザーのログイン体験を向上させました。

🔖 主な変更点以外に追加で変更された部分はありますか?

  • ログイン機能に必要なinterfaceを追加し、ローディングgifを追加しました。

🙏🏻 Reviewerに特に見ていただきたい部分はありますか?

  • Recoilを使った状態管理ロジックとGoogleソーシャルログインロジックでより良いセキュリティ関連の処理方法があれば教えてください。

🩺 このPRでテストや検証が必要な部分はありますか?

  • Recoilの状態管理ロジックが正常に動作していることを確認する必要があります。

📚 関連するIssueやTrello、ドキュメント

🖥 作動する様子

스크린샷 2024-04-19 오후 2 49 45

📌 PRを行う際の注意点

  • Reviewerはコードレビュー時に良いコードの方向性を示しますが、コード修正を強制することはありません。
  • Reviewerは良いコードを見つけた場合、賞賛と励ましを惜しみません。
  • レビューは特別なケースでない限り、Reviewerに指定された時点から3日以内に行ってください。
  • コメント作成時にPrefixにP1、P2、P3を書いていただくと、Assigneeがより明確にコメントに対して対応することができます。
    • P1 : 必ず反映してください (Request Changes) - 問題が発生したり、脆弱性が発見されたケースなど。
    • P2 : 反映を積極的に検討していただければと思います (コメント)。
    • P3 : こんな方法もあるんじゃないかな~などの些細な意見です (Chore)。

dorimu0 added 5 commits April 19, 2024 14:12
Add recoil library

Related issue: YJU-OKURA#105
- Create  RecoilRootContainer component and apply common layout
- Add user and token state required for login
- Add the required interface for recoilRootContainer

Related issue: YJU-OKURA#104
- Added Google Social Login API.
- Create component to perform redirect and login success handle API requests on successful login.
- Create a function to add a token to the axios header on successful login.
- Create function to reissue token use refresh token when an access token expires

Related issue: YJU-OKURA#104
- Define user information, tokens, and login-data interface
- Manage interfaces in the index file

Related issue: YJU-OKURA#104
- Add loading GIF file
- Manage gif with index file

Related issue: 104
@dorimu0 dorimu0 added ⚙ Setting 개발 환경 세팅 🎨 Html&css 마크업 & 스타일링 📬 API 서버 API 통신 ✨ Feature 기능 개발 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 labels Apr 19, 2024
@dorimu0 dorimu0 requested a review from a team April 19, 2024 05:51
@dorimu0 dorimu0 self-assigned this Apr 19, 2024
@Lainari
Copy link
Collaborator

Lainari commented Apr 20, 2024

P2 : 로그인 로직에서 주석처리 된 부분이 해결된다면 바로 PR 반영해주시길 바랍니다

@Lainari Lainari merged commit 31efe26 into YJU-OKURA:main Apr 20, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📬 API 서버 API 통신 ✨ Feature 기능 개발 🎨 Html&css 마크업 & 스타일링 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 ⚙ Setting 개발 환경 세팅
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants