Skip to content

[wip] タスクのコメントを取得できる api を叩くメソッドを追加#265

Merged
rikunosuke merged 3 commits intomainfrom
feature/264-get-comment-api-method
Mar 4, 2026
Merged

[wip] タスクのコメントを取得できる api を叩くメソッドを追加#265
rikunosuke merged 3 commits intomainfrom
feature/264-get-comment-api-method

Conversation

@daisuke-miyazoe
Copy link

@daisuke-miyazoe daisuke-miyazoe commented Mar 3, 2026

サマリ

概要、背景

タスクに紐づくコメントを取得する API メソッド get_comments を追加する。

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

対応内容

やったこと

  • Client クラスに get_comments メソッドを追加
    • project(プロジェクト名)と task_id(タスクID)を指定してコメント一覧を取得
    • offset / limit によるページネーションに対応
    • limit が 1000 を超える場合は FastLabelInvalidException を送出

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

  • なし

UI/UX

before

after

テスト

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

  • get_comments(project, task_id) でコメント一覧が取得できる
  • offset / limit を指定してページネーションが動作する

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

  • 既存の get_histories 等の周辺メソッドが正常に動作する

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

  • limit に 1001 以上を指定した場合に FastLabelInvalidException が発生する

関連リンク

補足

スクリーンショット 2026-03-03 15 10 24 スクリーンショット 2026-03-03 15 08 53

soymd
soymd previously approved these changes Mar 3, 2026
Copy link

@soymd soymd left a comment

Choose a reason for hiding this comment

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

編集しているファイルが巨大でびびりましたが、動作は問題ないと思います

@naoyamada-fastlabel
Copy link

naoyamada-fastlabel commented Mar 3, 2026

Reviewersにアサインされましたが宛先ミスですかね

@daisuke-miyazoe daisuke-miyazoe removed the request for review from naoyamada-fastlabel March 4, 2026 00:56
@daisuke-miyazoe
Copy link
Author

@naoyamada-fastlabel
すみません!
小山田さんをレビュアーに入れようと思って間違えてしまいました!
レビュアーから外しました

@rikunosuke
Copy link
Contributor

@daisuke-miyazoe @soymd
get_comments だと小山田さんに頼んでいるプロジェクトのコメント取得と名称が衝突しそうなので、そこを確認してください!
また、readme.md や sample.py などの作成をお願いします!

@rikunosuke rikunosuke changed the title タスクのコメントを取得できる api を叩くメソッドを追加 [wip] タスクのコメントを取得できる api を叩くメソッドを追加 Mar 4, 2026
@daisuke-miyazoe
Copy link
Author

daisuke-miyazoe commented Mar 4, 2026

@rikunosuke
小山田さんと相談してget_task_commentsに変更しました。
また、readme.md や sample.py などの作成は aad9eb3 であってますでしょうか?
ご確認お願いします。

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 5cb65b9 into main Mar 4, 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.

4 participants