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

企業一覧の各企業の「メンター」「アドバイザー」「現役生」の並びに「卒業生」を追加 #8189

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sugiwe
Copy link

@sugiwe sugiwe commented Nov 13, 2024

Issue

概要

企業一覧の各企業の「メンター」「アドバイザー」「現役生」の並びに、「卒業生」も表示するようにしました。

変更確認方法

  1. feature/feature/add_graduate_to_each_company_in_the_companies_pageをローカルに取り込む
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
  3. 任意のアカウントでログイン
  4. 企業一覧ページ を開き、各企業に「卒業生」の枠が追加されていることを確認
  5. 特に「BatsuBatsu Inc」は卒業生が1名入っておりクリックできるようになっているので、クリックして「BatsuBatsu Inc」の企業ページで卒業生がソートされた状態になっていることを確認

Screenshot

変更前

20241113c

変更後

20241113d   「BatsuBatsu Inc」の「卒業生1」をクリックした先の画面↓ 20241113h

@sugiwe sugiwe requested a review from mousu-a November 13, 2024 08:44
@sugiwe
Copy link
Author

sugiwe commented Nov 13, 2024

@mousu-a
お疲れさまです!
お手隙の際にこちらのレビューをお願いできますでしょうか🙏
ご都合が合わなければ遠慮なくおっしゃってください、どうぞよろしくお願いいたします!

@sugiwe sugiwe marked this pull request as ready for review November 13, 2024 08:47
@sugiwe sugiwe changed the title 企業一覧の各企業に「卒業生」を追加 企業一覧の各企業の「メンター」「アドバイザー」「現役生」の並びに「卒業生」を追加 Nov 13, 2024
@mousu-a
Copy link
Contributor

mousu-a commented Nov 13, 2024

@sugiwe
レビュー依頼ありがとうございますー!
ぜひ引き受けさせていただきます!
明後日くらいにはお返し出来ると思いますのでお待ち下さい🙏

@sugiwe
Copy link
Author

sugiwe commented Nov 13, 2024

@mousu-a
どうもありがとうございます!
明後日くらいにお戻しいただける旨も承知しました、多少後ろ倒しになっても全く問題ございませんので、どうぞよろしくお願いいたします🙏

Copy link
Contributor

@mousu-a mousu-a left a comment

Choose a reason for hiding this comment

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

@sugiwe
レビューお待たせいたしました!
コード良いと思います🙆

一応、忘れているだけかと思いますが右上のAssigneesを自分に設定しておくと良い感じです〜🙆

https://bootcamp.fjord.jp/pages/443#pr-%E3%82%92%E4%BD%9C%E6%88%90%E3%81%97%E3%81%9F%E3%82%89%E8%87%AA%E5%88%86%E3%82%92%E3%82%A2%E3%82%B5%E3%82%A4%E3%83%B3%E3%81%99%E3%82%8B

こちらからはApproveとさせていただきます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants