Skip to content

ワークスペースユーザーAPI公開対応#277

Merged
rikunosuke merged 5 commits into
mainfrom
feature/11296-add-workspace-user-api
Jun 22, 2026
Merged

ワークスペースユーザーAPI公開対応#277
rikunosuke merged 5 commits into
mainfrom
feature/11296-add-workspace-user-api

Conversation

@daisuke-miyazoe

@daisuke-miyazoe daisuke-miyazoe commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

サマリ

概要、背景

公開 API(アクセストークン認証)の Workspace User エンドポイントに、Python SDK を対応させる。

追加された API:

  • GET /workspaces-users — 内部ワークスペースユーザー一覧
  • POST /workspaces-users/internal-users — ユーザー追加(slug はサーバー側で自動生成)
  • PUT /workspaces-users/internal-users/{id} — role・モジュール権限の更新
  • DELETE /workspaces-users/internal-users/{id} — ユーザー削除

あわせて、招待・更新時にモジュール権限(functionResourcePermissions)を
付与・同期する対応も含む。

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

N/A(機能追加)

対応内容

やったこと

  • fastlabel/__init__.pyClient に 4 メソッドを追加
    • get_workspace_users(keyword, offset, limit) — レスポンスに functionResourcePermissions を含む
    • create_workspace_user(name, email, language, role, modules=None)
    • update_workspace_user(user_id, role=None, modules=None)
    • delete_workspace_user(user_id)
  • モジュール権限 modules"annotation" | "modelDev" | "dataset")に対応
    • create: 指定時のみ付与
    • update: None=変更なし / []=全剥奪 を is not None で区別して送信
  • README.md に Workspace User セクションを追加(引数・レスポンス例・modules の挙動)
  • tests/test_workspace_user.py を新規追加(endpoint / params / payload の検証、11 ケース)

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

UI/UX

before

N/A(SDK のため UI なし)

after

N/A

テスト

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

  • tests/test_workspace_user.py が全件パス(pytest tests/test_workspace_user.py → 11 passed)
  • 各メソッドが正しい endpoint・params・payload を組み立てることを確認

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

  • 既存テストスイートがパス(pytest

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

関連リンク

補足

@daisuke-miyazoe daisuke-miyazoe force-pushed the feature/11296-add-workspace-user-api branch from 94d2854 to 22dbd9c Compare June 15, 2026 09:41

@rikunosuke rikunosuke left a comment

Copy link
Copy Markdown
Contributor

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 0478dee into main Jun 22, 2026
9 checks 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