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

[ブログ] ブログ個別ページに最新の記事の一覧を10個表示したい #5213

Closed
machida opened this issue Jul 12, 2022 · 3 comments
Assignees

Comments

@machida
Copy link
Member

machida commented Jul 12, 2022

貼り付けた画像_2022_07_12_11_33

ここに最新の記事の一覧を10個表示。

各記事に表示するもの

  • 記事タイトル
  • サムネイル(記事に設定してあれば)
  • 作者のアイコン
  • 作者ID
@machida machida changed the title ブログに最新の記事の一覧を10個表示したい ブログ個別ページに最新の記事の一覧を10個表示したい Jul 12, 2022
@machida machida changed the title ブログ個別ページに最新の記事の一覧を10個表示したい [ブログ] ブログ個別ページに最新の記事の一覧を10個表示したい Jul 12, 2022
@komagata komagata added the 2 label Jul 13, 2022
@yocaji
Copy link

yocaji commented Jul 18, 2022

@machida @komagata
ブログの最新記事の要件について質問させてください。

当初は「publidhed_atに日時が入っている、かつWIPではない」記事を公開中の記事と判定して、publidhed_atが新しい順に並べる、という処理をイメージしていましたが、テストケースや過去のプルリクを調べてみるとpublidhed_atが空でもあえて公開済みの扱いにしていると分かりました。
また、ブログ記事一覧ページでソートに使われているのもpublished_atではなくcreated_atのようでした。

articles = Article.with_attached_thumbnail.includes(user: { avatar_attachment: :blob }).order(created_at: :desc).page(params[:page])

そこで、下記2点について確認させていただきたいです。

  1. 公開中の記事の判定は次のどちらを採用するべきでしょうか
    • publidhed_atに日時が入っている、かつWIPではない
    • WIPではない
  2. 最新記事のソートにはどちらの列を採用するべきでしょうか
    • published_at
    • created_at

お手数ですがご確認のほどよろしくお願いします🙇‍♂️

参考にしたIssue, PR

#4200
#3730

@yocaji
Copy link

yocaji commented Jul 20, 2022

2022-07-20 チーム開発ミーティングにて駒形さん、町田さんから以下の通り回答いただいた。

  • 公開中の記事の判定:WIPかどうか
  • ソート:published_at
    • 一覧画面のソートもpublished_atに変更する

基本的な方針として、最新記事のソートと一覧画面のソートは一致させる。
閲覧者にとってはpublished_atによるソートの方が自然なため、published_atを採用する。

@yocaji
Copy link

yocaji commented Aug 4, 2022

本番環境で動作確認できましたので、クローズいたします。

@yocaji yocaji closed this as completed Aug 4, 2022
@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants