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

プロフィール写真 #361

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

Regulus0811
Copy link
Collaborator

@Regulus0811 Regulus0811 commented Nov 14, 2024

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

  • LINEログインにおいて、初回ログイン時にLINEから提供されない場合に、デフォルトのプロフィール画像を特定の画像に置き換える処理を実装しました。これにより、プロフィール画像が設定されていないユーザーにも視覚的一貫性が保たれます。

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

  • LINEログイン初回時にプロフィール画像が提供されない場合、特定のデフォルト画像がユーザーのプロフィール画像として設定されるようにしました。
  • デフォルト画像のURLまたはパスを指定し、データベースに保存されるようにしました。

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

  • 特にありません。

🙏🏻 レビュアーに特に見ていただきたい部分はありますか?

  • プロフィール画像が提供されていない場合に、適切にデフォルト画像が設定されるかをご確認ください。
  • デフォルト画像がユーザー体験に適しているか、視覚的な一貫性が保たれているかも確認いただければと思います。

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

  • 初回ログイン時にプロフィール画像が設定されていないケースをシミュレートし、デフォルト画像が正しく表示されるかテストをお願いします。

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

  • 特に関連するIssueやドキュメントはありません。

🖥 作動する様子

  • デフォルト画像が設定されたプロフィール画面のスクリーンショットを提供します。

📌 PRを行う際の注意点

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

…t provide one

Replaced missing profile images with a specific default image for users logging in via Line for the first time.

Related issue: YJU-OKURA#349
@github-actions github-actions bot requested a review from Z00One November 14, 2024 06:06
@Regulus0811 Regulus0811 requested a review from yuminn-k November 14, 2024 06:06
@Regulus0811 Regulus0811 added ✨ Feature 기능 개발 📬 API 서버 API 통신 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 🆗 safe CI workflow 동작 관련 🥰 Accessibility 웹접근성 관련 labels Nov 14, 2024
@Regulus0811 Regulus0811 linked an issue Nov 14, 2024 that may be closed by this pull request
3 tasks
@Regulus0811 Regulus0811 merged commit e55c3b4 into YJU-OKURA:main Nov 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🥰 Accessibility 웹접근성 관련 📬 API 서버 API 통신 ✨ Feature 기능 개발 ⏰ P2 활발하게 고려해야 하는 이슈나 개선사항 🆗 safe CI workflow 동작 관련
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

LINEログインの実装
2 participants