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

attendanceページのエラー修正とLINEソーシャルログイン機能の追加、APIのアップデート #192

Merged
merged 5 commits into from
Nov 15, 2024

Conversation

dorimu0
Copy link
Contributor

@dorimu0 dorimu0 commented Nov 15, 2024

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

  • attendanceページで発生したエラーを修正し、ユーザーインターフェースを改善してユーザーエクスペリエンスを強化します。
  • 既存のGoogleソーシャルログインに追加的にLINEソーシャルログインを追加してソーシャルログインオプションを拡張します。
  • API関数ファイルでURL変更及びデータ返却形式を最新規格に合わせて修正して安定性を確保し、既存コードとの互換性を維持します。

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

  • attendanceページのエラー修正とユーザーインターフェース改善し、LINEソーシャルログインを追加及び関連API、ページ修正及びAPI関数ファイル内のURLとデータ返却形式を修正しました。

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

  • なし

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

  • LINEソーシャルログイン追加による認証フローで例外が発生しないか、API関数ファイルでURLとデータ返却形式の修正がシステム全体に影響を与えないことを確認していただければ幸いです。

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

  • LINEソーシャルログインと既存のGoogleログイン機能のテストが必要です。

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

🖥 作動する様子

  • なし

📌 PRを行う際の注意点

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

- Fixed layout inconsistencies and fixed alignment issues in the sidebar.
- Adjusted CSS settings for a consistent design across screen sizes.

Related issue: YJU-OKURA#71
- Applying the `hostname: '**'` setting because we don't think there will be any exceptions

Related issue: YJU-OKURA#189
- Reflected changes to some API endpoint URLs
- Modify your code to accommodate the changed data return format

Related issue: YJU-OKURA#190
- Added Line social login in addition to existing Google social login
- Fixed API and page related to Line Social Login

Related issue: YJU-OKURA#104
- Fix errors on the attendance page
- Add an attendance interface

Related issue: YJU-OKURA#191
@dorimu0 dorimu0 added 🎨 Html&css 마크업 & 스타일링 🐞 BugFix Something isn't working 📬 API 서버 API 통신 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 🆗 safe CI workflow 동작 관련 labels Nov 15, 2024
@dorimu0 dorimu0 requested a review from a team November 15, 2024 06:31
@dorimu0 dorimu0 self-assigned this Nov 15, 2024
Copy link
Collaborator

@Lainari Lainari left a comment

Choose a reason for hiding this comment

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

👍

@Lainari Lainari merged commit 2f08c09 into YJU-OKURA:main Nov 15, 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 통신 🐞 BugFix Something isn't working 🎨 Html&css 마크업 & 스타일링 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 🆗 safe CI workflow 동작 관련
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants