Skip to content

feat: get_project_comments メソッドを追加#269

Merged
rikunosuke merged 6 commits intomainfrom
feature/issue10863-project-comments-sdk
Mar 18, 2026
Merged

feat: get_project_comments メソッドを追加#269
rikunosuke merged 6 commits intomainfrom
feature/issue10863-project-comments-sdk

Conversation

@soymd
Copy link
Contributor

@soymd soymd commented Mar 17, 2026

サマリ

概要、背景

fastlabel/fastlabel-application#10863 対応のSDK実装。プロジェクト単位でコメント一覧を取得できる get_project_comments メソッドを追加。

(不具合の場合のみ) 発生原因

対応内容

やったこと

get_project_comments メソッドを追加

client.get_project_comments(
    project="my-project-slug",
    status="approved",           # タスクのステータスフィルタ(省略可)
    external_status="approved",  # タスクの外部ステータスフィルタ(省略可)
    tags=["tag1", "tag2"],       # タグフィルタ(省略可)
    issue_category_id="uuid",    # イシュー分類フィルタ(省略可)
    offset=0,                    # ページネーション(省略可)
    limit=100,                   # 最大1000(省略可)
)

API側の変更(fastlabel-application PR #10927)に対応。taskId を省略してプロジェクト全体のコメントを取得する。

やれていないこと、妥協点

UI/UX

before

after

テスト

変更の意図に沿った基本動作が確認できている

  • get_project_comments(project="slug") でコメント一覧が取得できる
  • フィルタパラメータが正しくAPIに渡される

関連する既存機能にデグレがないことを確認

  • 既存の get_task_comments に変更がないこと

エッジケースや例外パターンの動作を確認

  • limit > 1000 で例外が発生すること

関連リンク

  • fastlabel/fastlabel-application#10927

補足

- プロジェクト単位でコメント一覧を取得するSDKメソッドを追加
- フィルタパラメータ: status, external_status, tags, issue_category_id
@soymd soymd self-assigned this Mar 17, 2026
@soymd soymd marked this pull request as ready for review March 17, 2026 06:53
@soymd soymd requested a review from rikunosuke March 17, 2026 09:39
Copy link
Contributor

@rikunosuke rikunosuke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rikunosuke rikunosuke merged commit 3a9fddd into main Mar 18, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants