bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

event.vue, events.vueをVueMounterに対応させる

Open komagata opened this issue 2 years ago • 1 comments

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] avatar Aug 30 '22 01:08 github-actions[bot]

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] avatar Nov 17 '22 01:11 github-actions[bot]

reactへ書き換えようとして気づいたのですが、event.vue, events.vueは他のvueのコンポーネントやライブラリを使って依存しているのでそのままの書き換えが出来ません。

今回のevent関連のコンポーネントが依存しているのはPagerとLoadingListPlaceholder、UserIconです。LoadingListPlaceholderとUserIconは簡単にreactで書き直せると思いますが、Pagerは外部のライブラリを使っているため置き換えが難しいです。

個人的にはvueraのようなreact内でvueを使えるようにするライブラリを使ってreactへの置き換えが進むまで共存しやすいようにするのが良いと思うのですがどうでしょうか?

dowdiness avatar Dec 07 '22 09:12 dowdiness

@dowdiness Vueは使わなくなるので全部Reactに書き換えるようにお願いします。 コンポーネントの分け方はVueの時の設計を考慮しなくて良いです。

Pagerなどは管理画面の企業一覧でもReactで行なっているので参考にしてみてください。(Pagerのコンポーネントはこちらを使えると思います。)

管理画面の企業一覧 /admin/companies

komagata avatar Dec 07 '22 12:12 komagata

@komagata なるほどーvueで全部書き換える予定だったんですね。 Pagerのreactコンポーネントがあるのには気付かなかったですが、途中で気付いてcompaniesのコンポーネントを参考にこのコンポーネントを使って書くようにしました。

dowdiness avatar Dec 08 '22 06:12 dowdiness

マージされたのでクローズします

dowdiness avatar Jan 06 '23 04:01 dowdiness