管理者だけでなくメンターもプラクティスの編集をできるように変更

[makiichikawa]

ref #3052

メンターがプラクティスを修正できるように対応いたしました。

対応内容

1. プラクティス個別ページの編集ボタンをメンターでも表示できるようにした

変更前

変更後

2. 編集、更新処理前のログインチェックを管理者またはメンターでログインしているかに変更

[makiichikawa]

@gentamura
こちらのレビューお願いできますでしょうか？
お盆休み明けにお手すきの際にお願いできればと思っております🙇🏼‍♂️

[gentamura]

@makiichikawa [nits] 要件外ですが、同じ領域の部分なので、 current_user.mentor? || current_user.admin? を current_user.admin_or_mentor?すると良いなーと思いました！

bootcamp/app/views/practices/show.html.slim

Lines 138 to 155 in fa03f73

	- if current_user.mentor? || current_user.admin?
	section.a-card
	header.card-header
	h2.card-header__title
	= Practice.human_attribute_name :memo
	.practice-content__body.is-memo
	.js-markdown-view.js-target-blank.is-long-text
	= @practice.memo
	section.a-card
	header.card-header
	h2.card-header__title 管理者・メンター用メニュー
	footer.card-footer
	.card-main-actions
	ul.card-main-actions__items
	li.card-main-actions__item
	= link_to edit_practice_path(@practice), class: 'a-button is-md is-secondary is-block' do
	i.fas.fa-pen
	| 編集

[gentamura]

LGTM w/ nits 👌

[makiichikawa]

@gentamura
レビューありがとうございました。
メンター向けメモの枠と管理者メニューの表示条件が一緒になったという点では、要件外ではないのかなと思い、修正しました。
ご指摘いただき、ありがとうございました🙇🏼‍♂️

@komagata
こちらのレビューお願いできますでしょうか。よろしくお願いいたします🙇🏼‍♂️

[komagata]

最後のassertの後にvisitでページ移動しても意味ないかも？

[makiichikawa]

@komagata

プラクティスのページをvisitして、タイトルが更新されているか確認するように変更しました。
お手数をおかけしますが、再度確認お願いいたします🙇🏼‍♂️

[komagata]

確認漏れていて申し訳ないです。確認しました、OKですー🙆‍♂️