search

fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

未ログイン状態のトップ画面にGitHubのリンクを追加 #8161

Open hagiya0121 opened 1 month ago

hagiya0121 commented 1 month ago

Issue

ログイン前にGitHubへのリンクが欲しい #8159

概要

未ログイン状態のトップ画面にリポジトリリンクを追加しました。

変更確認方法

  1. feature/add-github-linkをローカルに取り込む
  2. 'foreman start -f Procfile.dev 'でローカルサーバーを立ち上げ
  3. 未ログイン状態でトップ画面にアクセスしてフッター部分にリンクが追加さ れていることを確認 http://localhost:3000/

    Screenshot

変更前

image

変更後

227f7ed549bd662a6b2bf73312798d45a3bbe16f0c4d42506de71feb56b2d4e9

hagiya0121 commented 1 month ago

@kurumadaisuke お疲れ様です。 こちらのPRのレビューをお願いしたいです🙏 ご都合が悪いときは遠慮なくおっしゃってください🙇

kurumadaisuke commented 4 weeks ago

@hagiya0121 すいません、ちょっと家の都合でバタバタしていまして別の方でも大丈夫でしょうか？🙇‍♂️

hagiya0121 commented 4 weeks ago

@kurumadaisuke 全然大丈夫です！

hagiya0121 commented 4 weeks ago

@mousu-a お疲れ様です。 こちらのPRのレビューをお願いしたいです🙏 ご都合が悪いときは遠慮なくおっしゃってください🙇

mousu-a commented 4 weeks ago

@hagiya0121 お疲れ様です！ レビュー了解です〜🙆 今週にはお返しできると思います！それでよろしければ引き受けさせていただきます〜

hagiya0121 commented 4 weeks ago

@mousu-a いつでも大丈夫です！よろしくお願いします🙏

hagiya0121 commented 3 weeks ago

@mousu-a レビューありがとうございます！指摘いただいたこと、2点とも知らなかったことなのですごい参考になりました🙏

hagiya0121 commented 3 weeks ago

@komagata メンバーのレビューが完了したのでレビューお願いします🙏

machida commented 3 weeks ago

Github → GitHub ですー

hagiya0121 commented 3 weeks ago

@machida ありがとうございます🙏修正しておきました。

mousu-a commented 1 week ago

@komagata すみません、コメントの意図がわからなかったのですが、

https://github.com/fjordllc/bootcamp/issues/8077#issue-2533716487

こちらのissueと同じようにtarget: '_blank', rel: 'noopener'をつけてほしい、ということでしょうか？

hagiya0121 commented 1 week ago

@mousu-a 質問タイムでお聞きしたところmousuさんがレビューで指摘したようにコミットをまとめてほしいとのことでした🙇