Skip to content

Conversation

@tacorice
Copy link
Contributor

@tacorice tacorice commented Aug 14, 2023

関連PR

・アプリケーション本体:
  https://github.com/fastlabel/fastlabel-application/pull/4288

(記載内容は本体PRと同じ)

概要

・APIからプロジェクトのタグを取得できるようにしました。
 ・以下のパラメータが使用できます。
  ・keyword
  ・offset
  ・limit
・APIからプロジェクトのタグを削除できるようにしました。
 ・tag_idsのリストを渡して一括で削除します。

テスト

  • SDKを使用して、タグを取得できること
  • SDKを使用して、タグを削除できること

@tacorice tacorice requested a review from anegawa-j August 14, 2023 12:04
@tacorice tacorice changed the title [EVERSTEEL]タグ取得・タグ削除のsdk WIP: [EVERSTEEL]タグ取得・タグ削除のsdk Aug 14, 2023
endpoint = "projects/copy"
return self.api.post_request(endpoint, payload=payload)

# Project Tags
Copy link
Contributor

Choose a reason for hiding this comment

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

Project Tagsだと、ProjectTagの方を連想してしまいそうなので、別の命名が良さそう

Copy link
Contributor

Choose a reason for hiding this comment

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

Tagsとかで良さそうな気がする

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ありがとうございます。修正しました。

@tacorice tacorice changed the title WIP: [EVERSTEEL]タグ取得・タグ削除のsdk [EVERSTEEL]タグ取得・タグ削除のsdk Aug 15, 2023
limit: int = 100,
) -> list:
"""
Returns a list of project tags.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Returns a list of project tags.
Returns a list of tags.

projectはとった方が良いかも

Copy link
Contributor

@anegawa-j anegawa-j left a comment

Choose a reason for hiding this comment

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

LGTM

@anegawa-j anegawa-j merged commit 23c97f0 into main Aug 15, 2023
@anegawa-j anegawa-j deleted the feature/add-new-api-to-get-and-delete-tags branch August 15, 2023 04:49
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.

3 participants