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

ブログのサムネイルが最新記事に表示されるように修正 #7803

Merged
merged 1 commit into from
May 28, 2024

Conversation

Shrimprin
Copy link
Contributor

@Shrimprin Shrimprin commented May 26, 2024

Issue

概要

ブログにサンプルで用意されているサムネイルを設定すると、「最新記事」欄のブログのサムネイルが表示されない不具合を修正しました。

変更確認方法

  1. fix/add-invoice-section-to-profileをローカルに取り込む

  2. bin/setupを実行

  3. foreman start -f Procfile.devでローカルサーバを立ち上げ

  4. admin権限のあるユーザー(例えばpjord)でログイン

  5. /articles/newにアクセスして、下記の通りブログを作成する

    • サムネイルはサンプルから選択する
    スクリーンショット 2024-05-26 150733
    • その他の項目は適当に入力する
  6. ブログ作成後に遷移した詳細ページで下記を確認する

    • 「最新記事」欄に、作成したブログのサムネイルが表示されている

Screenshot

変更前

before

変更後

after

@Shrimprin Shrimprin self-assigned this May 26, 2024
@Shrimprin Shrimprin requested review from kurumadaisuke and nakamu-kazu222 and removed request for kurumadaisuke May 26, 2024 06:24
@Shrimprin
Copy link
Contributor Author

@nakamu-kazu222
お疲れ様です 🍵
急ぎではありませんので、もし可能であればレビューをお願いできますでしょうか? 🙏
よろしくお願いいたします。

@Shrimprin Shrimprin marked this pull request as ready for review May 26, 2024 13:11
@nakamu-kazu222
Copy link
Contributor

@Shrimprin

お疲れ様です!
レビューさせていただきます!
1週間以内にレビューさせていただく予定です!

@Shrimprin
Copy link
Contributor Author

@nakamu-kazu222
ありがとうございます!
お手数おかけしますがよろしくお願いいたします 🙏

@nakamu-kazu222
Copy link
Contributor

@Shrimprin

お疲れ様です!

動作、コードの確認を行いまして、1点、以下の内容を確認していただきたいです

  • 変更確認方法のブランチ名が誤っていると思われます
    • 誤:feature/add-invoice-section-to-profile
    • 正:fix/display-blog-thumbnail-on-latest-articles

上記以外は特に問題ありません!

@Shrimprin
Copy link
Contributor Author

@nakamu-kazu222
お疲れ様です!
ご指摘いただきありがとうございます。
変更確認方法に記載されているブランチ名を修正しましたので、再度ご確認をお願いできますでしょうか 🙏

@nakamu-kazu222
Copy link
Contributor

@Shrimprin

お疲れ様です!
問題ないです!Approveします!

@Shrimprin
Copy link
Contributor Author

@nakamu-kazu222
ありがとうございます!
今後のチーム開発でもよろしくお願いいたします~ 🙏

@Shrimprin Shrimprin requested a review from komagata May 28, 2024 13:42
@Shrimprin
Copy link
Contributor Author

@komagata
メンバーからapproveいただきましたのでレビューお願いできますでしょうか 🙏
よろしくお願いいたします。

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させていただきました。OKです~👌

@komagata komagata merged commit 71594c7 into main May 28, 2024
10 checks passed
@komagata komagata deleted the fix/display-blog-thumbnail-on-latest-articles branch May 28, 2024 20:28
@github-actions github-actions bot mentioned this pull request May 28, 2024
18 tasks
@Shrimprin
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