bootcamp
bootcamp copied to clipboard
fjordllc
user_mentor_memo.vueを削除し、htmlとjsでの実装に変更
Issue
- #7453
概要
user_mentor_memo.vueを削除し、htmlとjsでの実装に変更しました。
変更確認方法
- feature/convert-user-mentor-memo-vue-to-slimをローカルに取り込む
- ローカル環境を起動し、管理者ユーザーでログインする
- ユーザーの相談部屋ページなどの、ユーザーメモが確認できるページに移動し、ユーザーメモの機能が正常に動作していることを、mainブランチと比較して確認する
「ユーザーメモ」タブ確認項目：
- 「ユーザーメモ」タブを選択すると、保存されているユーザーメモの内容がmarkdownとしてレンダリングされた状態で表示され、「編集」ボタンが表示される
- 「編集」ボタンをクリックすると、「メモ」タブが選択された状態で、テキストエリア内に保存されているユーザーメモの内容が表示されており、編集可能な状態になっている
- 「プレビュー」タブを選択すると、一時的に編集が不可能になり、ユーザーメモの内容がmarkdownとしてレンダリングされた状態で表示される
- 「メモ」タブ内でユーザーメモを編集してから「キャンセル」ボタンをクリックすると、ユーザーメモの内容が編集前の状態に戻り、画面表示が1.の状態に戻る
- 「メモ」タブ内でユーザーメモを編集してから「保存」ボタンをクリックすると、画面表示が1.の状態に戻り、ユーザーメモの内容は編集後のものになっている
- ユーザーメモの内容が空の状態で保存すると、画面表示が1.の状態になるが、画像と一緒に「プラクティスメモはまだありません。」との表示が出る
- ページを再読み込みすると、「ユーザーメモ」タブ内では、最後に保存した状態のユーザーメモが表示される
Screenshot
変更前
提出物のページで「ユーザーメモ」タブを選択
「編集」ボタンをクリックし、「メモ」タブが選択された状態
「プレビュー」タブが選択された状態
ユーザーメモを空にして保存した直後の状態
変更後
提出物のページで「ユーザーメモ」タブを選択
「編集」ボタンをクリックし、「メモ」タブが選択された状態
「プレビュー」タブが選択された状態
ユーザーメモを空にして保存した直後の状態
@SuzukaHori お疲れ様です。 こちらのPRのレビューをお願いしてもよろしいでしょうか🙏 お手すきの際で構いませんので、よろしくお願いいたします🙇♂️
@taco-nantai お疲れさまです！ 現在別のレビューに取り組んでおり、そちらをお待たせしている状況のため、今回は別の方に依頼をしていただいてもよろしいでしょうか🙇♀️ レビュー状態がFjord Choiceにうまく反映されていなかったかもしれません💦申し訳ありませんが、よろしくお願いいたします。
@masyuko0222 お疲れ様です。 こちらのPRのレビューをお願いしてもよろしいでしょうか🙏 お手すきの際で構いませんので、よろしくお願いいたします🙇♂️
@taco-nantai 承知しました〜！ ただ僕がフロントのIssueをまだやってないので、ちょっと時間かかってしまうかもしれません😅(まだ予想も付きませんが)
@taco-nantai お疲れ様です！五月雨ですみません！ 今の所、遅くて来週中のレビュー対応になるかと思います。
上記で問題なければこのままレビュー対応をさせて頂きます🙇♂️ 諸々のお気遣いは不要ですので、スケジュール感がご都合と合うかご確認ください〜！
@komagata お疲れ様です。 開発チームのレビューでmasyuko0222さんから https://github.com/fjordllc/bootcamp/pull/7470#discussion_r1518847585 で指摘をいただいた点に関して、問題がないか確認をお願いしたいです。 お忙しいところ恐縮ですが、よろしくお願いします。
@komagata コメントありがとうございます! 個人的には、ユーザーから見た挙動は変化しておらず、画面上に表示されていない要素が開発者ツールで見えているという点についても特に問題がないと考えたため、変更後の状態で問題ないと考えています。 その上で、現状の仕様に何か問題があれば、ご教授いただきたいです🙇♂️
@taco-nantai これってDevToolsを使えばメンター向けのユーザーメモが見れてしまうということでしょうか？ だとしたら非常にまずいのでどうやっても見えないようにしたいです。
理由としては例えば本人とメンター間でだけ内内に話してくれた内容（メンタルの病気を抱えていて、対応方法に留意点があるなど）を書く場所だからです。
@komagata コメントありがとうございます!
これってDevToolsを使えばメンター向けのユーザーメモが見れてしまうということでしょうか？
いいえ、違います。 mentorユーザーとadminユーザー以外のユーザーがメンター向けのユーザーメモを見れてしまうということはありません。 ユーザーメモを画面上で見ることのできるユーザー(mentorユーザーとadminユーザー)のみ、タブの切り替え等で一時的に画面上に表示されていないユーザーメモ関連の要素を、開発者ツールで見ることができるというだけです。 そのため、変更後の状態で問題ないと考えています。 その上で、現状の仕様に何か問題があれば、ご教授いただきたいです🙇♂️
@taco-nantai
mentorユーザーとadminユーザー以外のユーザーがメンター向けのユーザーメモを見れてしまうということはありません。
であれば大丈夫です。
conflictしているところがあるので修正をおねがいします～
@komagata コメントありがとうございます。
conflictしているところがあるので修正をおねがいします～
確認が漏れており失礼しました🙇♂️ conflictを修正したので、確認をお願いします。