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

企業研修申込の一覧ページと詳細ページを作成 #8252

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

hagiya0121
Copy link
Contributor

@hagiya0121 hagiya0121 commented Dec 16, 2024

Issue

概要

企業研修申し込みの一覧と詳細ページを作成しました。

変更確認方法

  1. feature/corporate-training-inquiriesをローカルに取り込む
    1. git fetch origin pull/8252/head:feature/corporate-training-inquiries
    2. git checkout feature/corporate-training-inquiries
  2. rails db:seedで初期データの追加
  3. foreman start -f Procfile.dev でローカルサーバーを立ち上げ
  4. 企業研修申し込み一覧ページにアクセス
  5. ページ上部に企業研修のタブがあることを確認
  6. ページネーションがページ上部と下部に二つ表示されていることを確認
  7. 初期データが受信日を基準に降順で並んでいることを確認
  8. 企業研修申し込みの詳細ページにアクセス
  9. 申し込み情報の各項目が表示されていることを確認

Screenshot

企業研修申し込み一覧画面

image

企業研修詳細画面

image

@hagiya0121
Copy link
Contributor Author

@machida
こちらのPRにデザインを適用していただきたいです🙏

@hagiya0121 hagiya0121 self-assigned this Dec 17, 2024
@machida
Copy link
Member

machida commented Dec 26, 2024

@hagiya0121 おまたせしました!!デザインを入れましたので、ご確認お願いします🙏

@hagiya0121 hagiya0121 force-pushed the feature/corporate-training-inquiries branch from a11907c to bfae74e Compare December 28, 2024 11:41
@hagiya0121 hagiya0121 marked this pull request as ready for review December 29, 2024 06:50
@hagiya0121 hagiya0121 requested a review from ayu-0505 December 29, 2024 06:53
@hagiya0121
Copy link
Contributor Author

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

@ayu-0505
Copy link
Contributor

@hagiya0121
お疲れさまです🍵
レビュー了解しました!
ただ、年末年始は家庭の用事が多くほとんどPCを触るタイミングがないため、
1月6日から1週間を目処にお時間いただきたいです🙏
それでよろしければぜひお受けします🙇🏻‍♀️

@hagiya0121
Copy link
Contributor Author

@ayu-0505
それで大丈夫です。よろしくお願いします🙏

Copy link
Contributor

@ayu-0505 ayu-0505 left a comment

Choose a reason for hiding this comment

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

大変お待たせしております🙇🏻‍♀️
動作、コード確認ともにチェックいたしましたが、気になる点がないため私の方からはApproveとさせていただきます。

viewsにおけるcollectionオプション等、忘れている内容もあり勉強させていただきました🙏

@hagiya0121
Copy link
Contributor Author

@ayu-0505
お忙しい中、レビューありがとうございます🙇

@hagiya0121
Copy link
Contributor Author

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

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.

3 participants