日報にブックマーク機能を実装 by alto-I · Pull Request #2789 · fjordllc/bootcamp

alto-I · 2021-06-05T09:25:31Z

issue: #2658

概要

日報をブックマークできる機能を実装。

ブックマーク一覧ページへのリンク

ブックマーク一覧

ブックマーク一覧はvue.jsで実装。並び順はブックマークした順。

ブックマークボタン

日報の各個別ページにブックマークボタンを実装。

その他

後に提出物やQ&Aにもブックマーク機能を付けるためポリモーフィックで実装
自分の日報もブックマークできる

alto-I · 2021-06-12T05:18:31Z

@SeijiNumata
こちらお手隙の際レビューお願いできないでしょうか🙇

SeijiNumata

確認しました！
とても良いと思います！
一点だけ質問しましたのでご確認ください！
よろしくお願いいたします🙇🏻

SeijiNumata · 2021-06-12T09:00:48Z

app/javascript/bookmark.vue

+            a.a-user-name(:href='bookmark.authorUrl')
+              | {{ bookmark.author }}
+          .thread-list-item-meta__item(v-if='bookmark.modelName == "Report"')
+            time.a-date(:datetime='bookmark.reportedOn', pubdate='pubdate')


ここのpubdateについて質問です！
現在廃止されているようなのですが、何かに使われているのでしょうか？

https://www.osaka-kyoiku.ac.jp/~joho/html5_ref/pubdate_attr.php?menutype=2dtaldl01l02l03A0

ご指摘ありがとうございます！
廃止されているのを知りませんでした😅

alto-I · 2021-06-13T09:48:15Z

@SeijiNumata
ご指摘受けた点修正しましたので、再度ご確認お願いします！😄

SeijiNumata · 2021-06-14T08:06:27Z

@alto-I
確認しました！
LGTMです！修正ありがとうございました！🙇🏻

alto-I · 2021-06-14T10:44:40Z

@SeijiNumata
ご確認ありがとうございました！勉強になりました😄

alto-I · 2021-06-14T12:50:33Z

@komagata
こちらレビューよろしくお願いします！😄

komagata · 2021-06-15T04:41:26Z

app/controllers/api/bookmarks_controller.rb

+  def index
+    bookmarks = Bookmark.where(user: current_user).order(created_at: :desc)
+    @bookmarks = Kaminari.paginate_array(bookmarks).page(params[:page]).per(PEGER_NUMBER)
+    return unless params[:bookmarkable_id] && params[:bookmarkable_type]


この場合、APIはどういうレスポンスを返すかんじですかね？
Railsのエラーが出ないようにしたいっすね。

@komagata
お返事遅くなってすいません！
この場合は、bookmarkable_id と bookmarkable_typeに対応したブックマーク情報を返すようになっています。ブックマークしてあればその日報の情報、してなければbookmarks以下が空のJSONを返します。

返すAPIの例

{"bookmarks": [ {"id":1046770788, "modelName":"Report", "modelNameI18n":"日報", "author":"Komagata Masaki", "authorUrl":"http://localhost:3000/users/459775584", "url":"http://localhost:3000/reports/701335546", "title":"作業週1日目", "reportedOn":"2017年01月01日" } ] }

日報の個別ページでブックマークしているかの判定に使っています。

この部分は自分でも少し気になっていたのですがindexの部分は

ブックマーク一覧の取得

日報のページでのブックマークしているかの確認

の二つで使用しており、役割を二つ持たせている状態になっています。

ここは別のコントローラーを用意して分けた方がいいでしょうか？

UI上の二つの場所から使われていることは全く問題ないです〜。
考え方としてはUIとは全く切り離して考えて、このサービスの /api/bookmarks.jsonというAPIとしてどういう動き・データを返すのが相応しいかというのだけを考えれば良い感じです〜
（RESTの観点）

app/controllers/api/bookmarks_controller.rb

komagata

確認しました、OKですー🙆‍♂️

komagata · 2021-06-24T01:51:36Z

@alto-I @machida こちらデザインは必要ですかね？まだマージはしないでおきます〜

alto-I · 2021-06-28T02:47:08Z

@komagata
ご確認ありがとうございます！
RESTの観点を自分がまだちゃんと理解できていない部分がありましたので復習します 💪

alto-I · 2021-06-28T02:49:31Z

@machida @komagata
こちら一度質問タイムでmachidaさんにデザインの大まかな部分は伺っているのですが、念の為ご確認をお願いしたいです🙇

komagata · 2021-06-28T05:26:55Z

@alto-I

こちら一度質問タイムでmachidaさんにデザインの大まかな部分は伺っているのですが、念の為ご確認をお願いしたいです🙇

こちらどう言う意味でしょうか？
僕に対しての内容ですかね？僕が確認すべきはどんな点になりますか？

alto-I · 2021-06-29T03:43:08Z

@komagata

@alto-I

こちら一度質問タイムでmachidaさんにデザインの大まかな部分は伺っているのですが、念の為ご確認をお願いしたいです🙇

こちらどう言う意味でしょうか？
僕に対しての内容ですかね？僕が確認すべきはどんな点になりますか？

すいません、こちらmachidaさんにデザインの確認をして欲しいという意味で書きました。わかりづらくて申し訳ありません🙇

@machida
デザインのご確認よろしくお願いします！🙇

machida · 2021-06-29T09:18:47Z

@alto-I デザイン了解ですー🙆‍♂️

Co-authored-by: Masaki Komagata <komagata@gmail.com>

machida · 2021-06-30T14:05:39Z

デザイン入れました。
テスト待ち。

checkboxのセレクタをnameに変更ブックマーク一覧の編集labelにforを追加 Watch中一覧の編集checkboxのテストの修正待つ設定をbookmarkテストに追加 Bookmarkのテストに待つ設定を追加ボタンの表示が切り替わる待ちを追加したブックマークボタンの文言変更チェックボックスのテスト書き方を変更テスト修正

machida · 2021-07-01T15:55:38Z

テストが通りましたのでマージしますー

alto-I · 2021-07-04T08:29:39Z

@machida
デザインご確認&マージありがとうございます！
リリースノートまで書いていただいてすいません🙇

alto-I force-pushed the feature/create_bookmark branch 2 times, most recently from f7bed7a to 97b2d54 Compare June 10, 2021 10:11

alto-I marked this pull request as ready for review June 12, 2021 05:13

alto-I requested a review from SeijiNumata June 12, 2021 05:14

SeijiNumata reviewed Jun 12, 2021

View reviewed changes

alto-I self-assigned this Jun 12, 2021

alto-I force-pushed the feature/create_bookmark branch from ce23fc5 to 85f198e Compare June 14, 2021 10:44

alto-I force-pushed the feature/create_bookmark branch 2 times, most recently from 85f198e to 64dd3c0 Compare June 14, 2021 12:12

alto-I requested a review from komagata June 14, 2021 12:50

komagata requested changes Jun 15, 2021

View reviewed changes

komagata approved these changes Jun 24, 2021

View reviewed changes

alto-I assigned machida Jun 28, 2021

komagata mentioned this pull request Jun 29, 2021

QAにブックマーク機能がほしい #1722

Closed

alto-I added 5 commits June 30, 2021 11:24

Bookmarkモデルの作成

ec59354

bookmarkableモジュールの作成とreportモデルへの関連付け

42b408c

ルーティングにbookmarkを追加

2ab7573

コントローラーの作成とルーティングの修正

18cc97d

コントローラーのファイル名の修正と一覧ページを作成

b2efc89

alto-I and others added 16 commits June 30, 2021 11:24

初期データを作成＋周辺ファイルの修正

7ebbd87

一覧ページのためのjs,vueファイルを作成

457ae67

ブックマーク一覧ページを実装

86767d1

一覧ページにページングを実装

8a1a059

不要なメソッドの中身を削除

8a2cb73

ブックマークボタン周辺のファイルを作成

b726c45

ブックマークボタンを実装

baa6ddc

細かな部分の修正

2bf78fd

bookmarkにユニーク属性のバリデーションを追加

db73cc0

ブックマークリストに詳細情報を追加と変数名の修正

0ec7091

APIテストの追加とその周辺の修正

01accf0

ブックマークボタンのidの付与とモデル、システムテストの実装

c902992

不要な属性(pubdate)を削除

9c448ae

不要なbefore actionを削除

dfb9cb5

不要なルーティングを削除

579ad3c

createメソッドの修正

690fc3e

Co-authored-by: Masaki Komagata <komagata@gmail.com>

machida force-pushed the feature/create_bookmark branch from 2084437 to 5f00e22 Compare June 30, 2021 14:05

machida force-pushed the feature/create_bookmark branch from d98272a to ed3dc66 Compare July 1, 2021 15:44

machida merged commit 1903ef6 into main Jul 1, 2021

machida deleted the feature/create_bookmark branch July 1, 2021 15:55

github-actions bot mentioned this pull request Jul 1, 2021

Release 2021-07-05 05:08:38 #2883

Merged

18 tasks

djkazunoko mentioned this pull request May 6, 2023

ブックマーク一覧ページにページネーションを追加 #6459

Merged

tyrrell-IH mentioned this pull request Dec 2, 2025

定期イベントにブックマークボタンを追加 #9306

Merged

2 tasks

yokomaru mentioned this pull request Mar 16, 2026

datetime属性の形式を統一 #9717

Open

Conversation

alto-I commented Jun 5, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概要

ブックマーク一覧ページへのリンク

ブックマーク一覧

ブックマークボタン

その他

Uh oh!

alto-I commented Jun 12, 2021

Uh oh!

SeijiNumata left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

SeijiNumata Jun 12, 2021

Choose a reason for hiding this comment

Uh oh!

alto-I Jun 13, 2021

Choose a reason for hiding this comment

Uh oh!

alto-I commented Jun 13, 2021

Uh oh!

SeijiNumata commented Jun 14, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alto-I commented Jun 14, 2021

Uh oh!

alto-I commented Jun 14, 2021

Uh oh!

komagata Jun 15, 2021

Choose a reason for hiding this comment

Uh oh!

alto-I Jun 23, 2021

Choose a reason for hiding this comment

Uh oh!

komagata Jun 24, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

komagata left a comment

Choose a reason for hiding this comment

Uh oh!

komagata commented Jun 24, 2021

Uh oh!

alto-I commented Jun 28, 2021

Uh oh!

alto-I commented Jun 28, 2021

Uh oh!

komagata commented Jun 28, 2021

Uh oh!

alto-I commented Jun 29, 2021

Uh oh!

machida commented Jun 29, 2021

Uh oh!

machida commented Jun 30, 2021

Uh oh!

machida commented Jul 1, 2021

Uh oh!

alto-I commented Jul 4, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

alto-I commented Jun 5, 2021 •

edited

Loading

SeijiNumata left a comment •

edited

Loading

SeijiNumata commented Jun 14, 2021 •

edited

Loading

alto-I commented Jul 4, 2021 •

edited

Loading