Skip to content

Conversation

@idotta
Copy link
Owner

@idotta idotta commented Dec 31, 2025

Create get all tenants endpoint


Note

Introduces an Admin API surface and deprecates the old System (/sys) routes.

  • Rename and regroup endpoints from /sys to /admin via new AdminEndpoints; update routing, DI registrations, docs, sample UI/JS, and tests accordingly
  • Move system-related features into Idmt.Plugin.Features.Admin (create/delete tenant, grant/revoke/list user tenants, system info) and update MapIdmtEndpoints
  • Add GET /admin/tenants to list all tenants, returning TenantInfoResponse[] (now includes IsActive)
  • Validation and policy tweaks: enforce tenant identifier format (Validators.IsValidTenantIdentifier) in CreateTenant; forbid deleting default tenant; CreateTenant requires RequireSysUser, DeleteTenant now RequireSysAdmin; keep system/info behind RequireSysUser
  • Update seeding to use Admin create-tenant handler; minor cleanup in ForgotPassword and tests

Written by Cursor Bugbot for commit f14be17. This will update automatically on new commits. Configure here.

Create get all tenants endpoint
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

- Updated CreateTenant to improve identifier validation logic.
- Enhanced GetAllTenants and GetUserTenants to include IsActive status in the response, providing better visibility into tenant activity.
@idotta idotta merged commit c206431 into main Dec 31, 2025
2 checks passed
@idotta idotta deleted the sys-admin branch December 31, 2025 20:59
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