refresh() instance method on PromiseState
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript, react
調査の方向性
docs/api.md の PromiseState セクションから始め、ドキュメントに記載されている静的な refresh() メソッドと、この issue に示されている手動の refresh パターンを比較します。既存の PromiseState の実装と、存在する場合はそのテストを追跡し、インスタンスメソッドがどのように動作すべきかを判断します。props.userFetch.refresh() が要求された refresh の動作を提供し、PromiseState のドキュメントとテストでそれがカバーされれば完了です。
索引モデルが issue の本文から書いたものです。
説明
I'm new to the library but i've been using the manual refresh pattern from the readme a lot because each of my screens have a refresh button.
connect(props => {
const url = `/users/${props.userId}`
return {
userFetch: url,
refreshUser: () => ({
userFetch: {
url,
force: true,
refreshing: true
}
})
}
})(Profile)
Reading through the documentation for PromiseState i noticed that there is a refresh() static method but not an instance method. I think adding it as an instance method would be hugely helpful to a lot of people. It would reduce the above to:
connect(props => ({
userFetch: `/users/${props.userId}`,
})(Profile)
and then call the refresh using props.userFetch.refresh()
Am i missing something? It doesn't look like anyone has mentioned it before, do they just write the extra code?
- 主要言語
- JavaScript
- スター
- 3.4k
- フォーク
- 137
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
heroku/react-refetch のほかの issue
-
Support for react@17オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
heroku/react-refetch#258 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 45/100
heroku/react-refetch#243 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
heroku/react-refetch#228 · コメント 1 件 · リアクション 3 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
heroku/react-refetch#227 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
heroku/react-refetch#226 · コメント 1 件 ·
heroku/react-refetch の issue をすべて見る
似ている issue
-
Four problem loader browser tests fail on Windows対応中かも @Patriciaath999 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
juice-shop/juice-shop#3686 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ScottKirvan/GOMAC#56 ·
-
area:install bug needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
yetone/magpie#2406 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信