「参考書籍一覧」を非Vue化した

ham-cap commented 2 months ago

Issue

これまでVueで実装されていた参考書籍一覧のページ（/books）をHTMLによる実装に変更しました。

chore/change-books-list-from-vue-to-htmlをローカルに取り込む
http://localhost:3000/books にアクセス
参考書籍の一覧が表示されていることを確認する
プラクティスによる書籍の絞り込み機能が動作していることを確認する
- 例えば「OS X Mountain Lionをクリーンインストールする」を選択すると以下のようになります。

なお、非Vue化に伴い、絞り込みの方法を動的なDOMツリーの編集からサーバーへのリクエストによる絞り込みに変更しています。これにより、絞り込み機能を使用するとURLにpractice_idとsearch_termsというパラメータが含まれるようになりました。このうちsearch_termsについては絞り込みの選択肢をフリーワードで検索する機能のために必要なものとなります。たとえば、絞り込みのプルダウンの最上段にある検索窓に「sql」と入力してプラクティスの選択肢を検索したうえで、「sqlの基礎を理解する」を選択して書籍の絞り込みを行った場合のURLと画面表示は以下のようになりますので、併せてご確認いただければと思います。

Vueで実装された既存のページをHTML化したものなので、画面の表示そのものに変更はありません。

ham-cap commented 2 months ago

@kurumadaisuke お疲れ様です！こちらのPRについてレビューをお願いできますでしょうか🙏 特に急ぎではありませんのでお手隙の際で大丈夫です🙆‍♂️ お忙しいかと存じますので難しければ遠慮なくおっしゃってください👍

kurumadaisuke commented 2 months ago

@ham-cap はい！こちら確認させていただきます🙏 少々お時間いただければっと思いますがよろしくお願いいたします！

ham-cap commented 2 months ago

@kurumadaisuke お忙しい中ご対応いただきありがとうございました✨

@komagata チームメンバーからApproveいただきましたのでレビューをお願いいたします🙏

ham-cap commented 1 month ago

@komagata (CC: @machida ) 修正いたしましたのでご確認をお願いいたします。なお、ご指摘いただいた何もないdivについてですが、確認してみたところこのdivが無くてもデザイン崩れ等ありませんでしたので削除させていただきました🙏

machida commented 1 month ago

何もないdivの削除ありがとうございますー🙏

ham-cap commented 1 month ago

@komagata ご指摘いただいたリンクの箇所について修正いたしました！なお、直接ご指摘いただいたわけではないですが、aタグからlink_toへ置き換える場合と本質的に同じと考え、一箇所imgタグをimage_tagへ置き換えています。ご確認のほどよろしくお願いいたします🙏

ham-cap commented 2 weeks ago

@komagata インデント等修正いたしました！ご確認のほどよろしくお願いいたします🙏