reports.vueをreactに書き換える

[komagata]

#6039こちらで作られたapp/javascript/components/UnconfirmedLink.jsxを使う。

[lef237]

@komagata
お疲れ様です！

こちらのIssueについてご確認したいことがあります。🙏

「reports.vueをreactに書き換える」というのがこのIssueで取り組む内容だと思うのですが、reports.vueのReact化については、すでにこちらのPull Requestでpenoさんが取り組まれている最中でした。

個人の日報一覧をプラクティスで絞り込めるようにした by peno022 · Pull Request #6044 · fjordllc/bootcamp

そのため、 #6044 のPull Requestがmergeされるのを待ってから、そこに変更を加える（app/javascript/components/UnconfirmedLink.jsxを組み込む）べきでしょうか？

それとも、このissueに関してはアサインを解除して頂いて、別のissueを割り振ってもらったほうが良いでしょうか？

[lef237]

📝 2023年2月1日 ふりかえり・計画ミーティングよりメモ

こちらのissueについては、#6044 がmainブランチにmergeされてから取り組むことになりました。

[lef237]

@komagata
本番環境で確認したところ、ページネーション関係で条件分岐の漏れを発見致しました。

こちらのPull Request #6261 で修正・対応致しましたので、ご確認頂けると幸いです🙏

[komagata]

@lef237 PRありがとうございます。そちらの方をマージしました〜

[lef237]

こちら本番環境で確認できましたので、Close致します。