Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`
評価
調査の方向性
QuickbaseNet/Helpers/QuickbaseQueryBuilder.cs、QuickbaseNet/Models/Options.cs、QuickbaseNet/Requests/QuickbaseQueryRequest.csを読み、その後、既存のクエリビルダーのテストを確認してください。要求されたビルダーメソッドとオプションのシリアル化動作を実装し、どちらのメソッドも呼び出されない場合はOptionsをnullのままにしてください。dotnet test --filter FullyQualifiedName~QuickbaseQueryBuilderTestsを実行し、記載された3つの完了条件を確認してください。
索引モデルが issue の本文から書いたものです。
説明
The query endpoint takes skip and top options for paging. QuickbaseQueryRequest already has an Options property for them, but QuickbaseQueryBuilder never sets it. This is the paging item in ROADMAP.md.
What to do
- Add
Skip(int count)andTop(int count)toQuickbaseQueryBuilder. Each returns the builder, like the other methods. - In
Build(), setOptionsonly when one of them was called. Otherwise leave it null, as it is today. - In
Options, addDefaultValueHandling = DefaultValueHandling.Ignoreto theskipandtopproperties, so calling onlySkip(100)doesn't also send"top": 0. - Add XML doc comments to both methods.
QueryAllRecords is a separate roadmap item. Leave it out.
Where to look
QuickbaseNet/Helpers/QuickbaseQueryBuilder.cs(Build()is at line 86)QuickbaseNet/Models/Options.csQuickbaseNet/Requests/QuickbaseQueryRequest.cs, line 46
Done when
-
.Skip(100).Top(50)builds a request withOptions.Skip100 andOptions.Top50 - A request built without either has
Options == null -
JsonConvert.SerializeObjectof a request built with only.Skip(100)contains"skip":100and no"top"key - Tests for all three in
QuickbaseNet.UnitTests/Tests/QuickbaseQueryBuilderTests.cs(create it if it doesn't exist)
Testing it
dotnet test --filter FullyQualifiedName~QuickbaseQueryBuilderTests
- 主要言語
- C#
- スター
- 3
- フォーク
- 2
- 平均マージ
- 5時間 53分
- マージ済み PR(30日)
- 1
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
hazeliscoding/quickbase-net のほかの issue
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
Update the package metadata that still points at ducksoop対応中かも @hazeliscoding が 2 日前に担当しました。 オープン
hazeliscoding/quickbase-net#15 · 担当者 1 名 ·
hazeliscoding/quickbase-net の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
KnifeLemon/Filee#363 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
-
area-Native untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
dotnet/machinelearning#7837 ·
メンテナーはふだん 2 日以内に返信
-
[CI] .trgen-ignore required for several grammars.対応中かも @kaby76 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
antlr/grammars-v4#5073 ·
メンテナーはふだん 1 日以内に返信
-
[Enhancement]: Add a search box to the Time Zone dropdown対応中かも @idumlupinar が今日担当しました。 オープンStatus: New
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
dnnsoftware/Dnn.Platform#7579 ·
メンテナーはふだん 2 日以内に返信