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.

Sign up for GitHub

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

Jump to bottom

参考書籍ページで各書籍に紐付くプラクティスへのリンクを追加 #5025

Merged
merged 6 commits into from
Jun 24, 2022
Merged

参考書籍ページで各書籍に紐付くプラクティスへのリンクを追加 #5025

merged 6 commits into from
Jun 24, 2022

Conversation

SatoshiHaramura
Copy link
Contributor

@SatoshiHaramura SatoshiHaramura commented Jun 13, 2022 •
edited
Loading

Issue

概要

参考書籍ページにおいて、各書籍に紐付くプラクティスへのリンクを追加しました。

確認方法

  1. ブランチfeature/add-link-to-practice-lassociated-with-book-in-reference-booksをローカルに取り込む。
  2. bin/rails sでローカル環境を立ち上げる。
  3. 任意のユーザでログインし、/booksにアクセスする。

確認内容

  1. 書籍にプラクティスが紐付く場合
    1-1. プラクティス個別ページへのリンクが表示されていることを確認する。
    1-2. プラクティス個別ページへのリンクをクリックし、/practices/:idへアクセスできることを確認する。
  2. 書籍にプラクティスが紐付かない場合
    2-1 何も表示されないことを確認する。

変更前

Cursor_and__development__参考書籍___FJORD_BOOT_CAMP（フィヨルドブートキャンプ）

変更後

/booksにアクセスしたときの、参考書籍ページ

Cursor_and__development__参考書籍___FJORD_BOOT_CAMP（フィヨルドブートキャンプ）

　↓

プラクティス個別ページへのリンクをクリック後の、プラクティス個別ページ

Cursor_and__development__OS_X_Mountain_Lionをクリーンインストールする___FJORD_BOOT_CAMP（フィヨルドブートキャンプ）

@SatoshiHaramura SatoshiHaramura self-assigned this Jun 13, 2022
@SatoshiHaramura SatoshiHaramura marked this pull request as ready for review June 13, 2022 14:52
@SatoshiHaramura
Copy link
Contributor Author

SatoshiHaramura commented Jun 13, 2022 •
edited
Loading

@isshi-hasegawa
お疲れ様です。
レビューの依頼をさせて頂きたいです🙏
こちら(good first issue 攻略・その１「着手からレビュー依頼まで」)のドキュメントが分かりやすくて、とても助かりました✨ありがとうございます！
お手数ですが、宜しくお願い致します。

@machida
お疲れ様です。
本日の質問タイムでお聞きしたとおり、デザインの対応を依頼させて頂きます🙏
一旦、ul/liタグを入れてみましたが、適切がどうか自信が無いため、併せてご確認頂けますと幸いです。

isshi-hasegawa
isshi-hasegawa approved these changes Jun 13, 2022
View reviewed changes
Copy link
Contributor

@isshi-hasegawa isshi-hasegawa left a comment

Choose a reason for hiding this comment

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

@SatoshiHaramura
実装お疲れさまでした。
ドキュメントがお役に立ったようで良かったです〜〜
動作の方もコードの方も確認いたしました👍

@SatoshiHaramura
Copy link
Contributor Author

@isshi-hasegawa
レビューして頂きありがとうございました！

@SatoshiHaramura SatoshiHaramura requested a review from komagata June 14, 2022 00:40
@SatoshiHaramura
Copy link
Contributor Author

@komagata
お疲れ様です。
チームメンバーのレビューが通りましたので、お手すきの際にレビューをお願い致します🙏
あと、町田さんにデザインを入れて頂く必要があるのですが、こちらは依頼中となります。

komagata
komagata requested changes Jun 15, 2022
View reviewed changes
app/views/books/_book.html.slim Outdated
@@ -16,6 +16,12 @@
.card-books-item__description
.a-short-text
= simple_format(book.description)
ul
- if book.practices_books.exists?
Copy link
Member

Choose a reason for hiding this comment

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

bookから直接practicesが取れるかと思います〜

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@komagata
承知致しました。

bookから直接practicesを呼べるようにするために、併せて修正致します。

@SatoshiHaramura SatoshiHaramura force-pushed the feature/add-link-to-practice-lassociated-with-book-in-reference-books branch from b3f14ca to f6abd2e Compare June 16, 2022 02:05
@SatoshiHaramura
Copy link
Contributor Author

@isshi-hasegawa
お疲れ様です。
駒形さんとのレビューで、こちらのコメント(#5025 (comment)) を頂き、修正を行いましたので、再度レビューをお願いさせて頂きたいです🙏
お手数おかけします🙇‍♂️

@isshi-hasegawa
Copy link
Contributor

@SatoshiHaramura
この場合レビュー依頼は手戻りすることなく、komagataさんに再レビュー依頼お願いします〜
とは言え、komagataさんのご指摘は勉強になりました😅

@SatoshiHaramura
Copy link
Contributor Author

@isshi-hasegawa
この場合はレビュー依頼の手戻りは不要なのですね！承知致しました！

@komagata
お疲れ様です。
頂いていたコメント(#5025 (comment)) に対応する修正を行いましたので、再度レビューをお願い致します🙏

@machida
Copy link
Member

machida commented Jun 16, 2022

@SatoshiHaramura デザイン入れましたー

@machida machida removed their assignment Jun 16, 2022
@SatoshiHaramura
Copy link
Contributor Author

@machida
お疲れ様です。
デザインを入れて頂きありがとうございます！
ただlintが失敗しているように見受けられるのですが、これはどのように対処すればよいのかご存知でしょうか？

SatoshiHaramura
SatoshiHaramura commented Jun 23, 2022
View reviewed changes
app/views/books/_book.html.slim Outdated
Comment on lines 30 to 38
- book.practices.each do |practice|
li.tag-links__item
= link_to practice.title, practice_path(practice), class: 'tag-links__item-link'
- book.practices.each do |practice|
li.tag-links__item
= link_to practice.title, practice_path(practice), class: 'tag-links__item-link'
- book.practices.each do |practice|
li.tag-links__item
= link_to practice.title, practice_path(practice), class: 'tag-links__item-link'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@machida
お疲れ様です。
先日デザインを入れて頂いた際に、上記のコードが重複しているように見受けられるのですが、削除してもよろしいでしょうか？ご確認お願い致します🙏

Copy link
Member

Choose a reason for hiding this comment

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

@SatoshiHaramura すいません！デザイン 調整のために書いたものを消し忘れてました🙇‍♂️ 消しておきますー

@machida
Copy link
Member

machida commented Jun 23, 2022

@SatoshiHaramura 修正しましたー。多分、lintも通ると思いますー

@SatoshiHaramura
Copy link
Contributor Author

@machida
ご対応頂きありがとうございました！lintも通っておりました🙏

@komagata
お疲れ様です。
machidaさんにデザインを入れて頂き、lintも通りましたので、改めてレビュー依頼をさせて頂きます🙏
お手すきの際に宜しくお願い致します。

komagata
komagata approved these changes Jun 24, 2022
View reviewed changes
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 eb53de0 into main Jun 24, 2022
@komagata komagata deleted the feature/add-link-to-practice-lassociated-with-book-in-reference-books branch June 24, 2022 04:56
@github-actions github-actions bot mentioned this pull request Jun 24, 2022
Merged
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@machida machida machida approved these changes

@isshi-hasegawa isshi-hasegawa isshi-hasegawa approved these changes

@komagata komagata komagata approved these changes

Assignees

@SatoshiHaramura SatoshiHaramura

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@SatoshiHaramura @isshi-hasegawa @machida @komagata