Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Pass this.state.data to then(), catch(), andThen() and andCatch() functions for more powerful side effects

オープン
#227 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
機能追加
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
javascript, react
領域
frontend

調査の方向性

ファイルパスやテストパスは指定されていません。まず then()、catch()、andThen()、andCatch() の実装に加えて、直接マッピング関数を追跡し、ハンドラーの引数がどのように組み立てられるか、また state.data をどのように公開できるかを確認してください。これらのハンドラーとマッピングから現在のデータにアクセスして取得結果をマージし、楽観的更新をサポートでき、関連する動作がテストでカバーされていれば完了です。

索引モデルが issue の本文から書いたものです。

説明

Does it make any sense to have this.state.data available in those handlers? In my case, it's relevant for implementing an infinite scroller, where each fetch adds to the previous data and I need to somehow merge it.

Here's how that would look like:

const connections = {
  widgets: '/api/widget/?limit=20',
  loadMore: offset => ({
    loadingMore: {
      url: `/api/widget/?limit=20&offset=${offset}`,
      andThen: (res, meta, data) => ({
        widgets: { value: data.widgets.value.concat(res) },
      }),
    },
  }),
};

My current workaround is to pass a getData() function to loadMore() from the component, which works, but isn't nice.

On another note, I would like to have that data even in direct mapping functions (loadMore()above), so I can do things like optimistic updates.

And, 👍 for this very nice library, thank you for writing it!

主要言語
JavaScript
スター
3.4k
フォーク
137
PR マージ指標
30日以内にマージされた PR はありません

環境構築

このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

heroku/react-refetch のほかの issue

heroku/react-refetch の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。