natsuto6
natsuto6
@komagata @machida こちらのIssueについて質問させてください。 12/27(水)の開発MTGにて確認させていただきましたが、`generation-users.vue`をReact化するにあたり、関連するファイル全てをReact化しております。 その中に`pager.vue`が含まれており、このページネーションをReactで実現するために、[react-paginate](https://github.com/AdeleD/react-paginate)の使用はどうかと考えています。`pager.vue`のロジックを維持し、見た目も同じにするためには、`react-paginate`を使用するのが良いのではと考えました。スター獲得数、メンテナンスの頻度を見ても信頼できるかと判断しました。 このライブラリを使用する前に、ご意見を伺いたいと思いましたので何か他の推奨する方法や注意点がありましたら、教えていただけますと幸いです。
3/27(水)チーム開発MTG 結論、pagerのcomponentを使う https://github.com/fjordllc/bootcamp/pull/7170#discussion_r1536711474
本番環境で動作確認完了しましたのでcloseします
@machida generation-users.vueとそれに関連するファイルをReactに書き換えました。 以下の点について、本番環境と同様になるようスタイルの追加をお願いできますでしょうか。 もし自分が実装した箇所で不足しているところがあれば、教えていただきたいです。 お手数ですがご確認よろしくお願いいたします。 - ローカル環境 1. 最初と最後のページへのリンクを配置するため、`ReactPaginate`コンポーネントの前後にそれぞれのボタンを配置しています 2. ユーザーごとのアイテム、両サイドの余白が追加されないのでご確認いただきたいです - 本番環境
@machida 上記いかがでしょうか。お手数ですがご確認よろしくお願いいたします。
@machida [こちら](https://github.com/fjordllc/bootcamp/pull/7170#issuecomment-1978059996)ご対応いただきありがとうございます。 実装が思いつかなかったので別Issueとさせていただければと思います。
@yocchan-git お疲れ様です!こちらお手数ですがレビューをお願いできますでしょうか? ご確認よろしくお願いします🙏
@yocchan-git 早々にご確認いただきありがとうございます! フォロー機能修正しましたので再度ご確認お願いします🙏 指摘いただいた1点目について追加しました。 > ブランチ名はなんでもいいのですが、フィヨルドのルールに則った方がいいかなと こちら確かにそうかもしれないので以後気をつけます🙇♂️ ただ、Issueの参考ページだったり過去のプルリクエストとかを参考に作成しました👀
@yocchan-git ご指摘いただきありがとうございます!🙇♂️ ご指摘の2箇所の挙動を修正しました。お手数ですが再度ご確認をお願いいたします🙏
@yocchan-git 相当な時間かけてしまったので全然です💦 早々にご確認いただきありがとうございました〜! @komagata チームメンバーのレビューが通りましたので、お手数ですがレビューをお願い致します🙇♂️