bootcamp
bootcamp copied to clipboard
fjordllc
categories.vue, check-stamp.vueをVueMounterに対応させる
下記を参考にしてVueMounterを使ったマウント方法に変える。
- Vue Componentのマウント方法 · fjordllc/bootcamp Wiki
- announcementsをVueMounterに対応 by komagata · Pull Request #5113 · fjordllc/bootcamp
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。
@komagata おはようございます！ こちらのIssue進め方について２点確認したいことがあります。
categories.vue について
#5660 ですでにReact化されていました。そのため、今回はcheck-stamp.vueのみReact化しようと思います。
React化の方針
すでにapp/javascript/components/Reports.jsx と app/javascript/components/Report.jsx の中にスタンプを実装している箇所がありました。 useSWR を用いてスタンプに関するデータ取得を行い、その内容を出力しています。
そのコードを参考に実装を進めようと思います。重複する箇所が発生してしまいますが、共通化は必要であれば別Issueを立てて後日実施したいです。
以上です。確認のほどよろしくお願いします。
@kyokucho1989
https://github.com/fjordllc/bootcamp/pull/5660 ですでにReact化されていました。そのため、今回はcheck-stamp.vueのみReact化しようと思います。
おお、すみません、その通りで大丈夫です。Issueのタイトルからも抜いておきました〜。
React化の方針
すみません、こちら( check-stamp.vue )、もしかしたら @dowdiness さんが今作業中かもしれないので、そちらを確認してみてください〜
@komagata
さきほど確認しました。
check-stamp.vue についても、
#7216 このプルリクですでに実装しているとのことです。
こちらのIssueはクローズした方がよいでしょうか？
@komagata @kyokucho1989
先程このIssueについて確認を受けました。
- https://github.com/fjordllc/bootcamp/issues/5122
今こちらの自分の担当しているCommentコンポーネントに関係するIssueがあるか確認しました。
- https://github.com/fjordllc/bootcamp/issues/5118
- https://github.com/fjordllc/bootcamp/issues/5119
- https://github.com/fjordllc/bootcamp/issues/5134
この上記の３つのIssueは、Commentコンポーネントの中で使われているか依存関係があるため、先にReactコンポーネントに変換しないとCommentコンポーネントのreact化が出来ません。
なので既にこちらのPRや、手元にあるブランチの作業ではReactに書き換えてしまっています。
Issueにコメントしたり自分にアサインしたりして、他の人が作業しないように分かりやすくするべきでした。 二度手間になってしまいすいません!🙇
@dowdiness コメントありがとうございます。 いえ、僕もこちら見逃しておりました。僕も気づけたはずなのに残っていてすみません。
@kyokucho1989 調査したことを日報で読ませていただきました。 調査していただいて申し訳ありませんが、こちらはすでに @dowdiness さんの方で進んでいるのでなしにしてください。 ただ、すでに途中まで調査いただいているので1ポイント割り振らせていただいたので1ポイントゲットとしちゃってください〜