Skip to content

Conversation

@aviadl
Copy link
Member

@aviadl aviadl commented Jan 27, 2026

  • tests

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for configuring tenant role inheritance when creating/updating tenants (intended for sub-tenant setups), along with test updates.

Changes:

  • Extend management.tenant.create, createWithId, and update to accept/send a roleInheritance field.
  • Update tenant management tests to cover request payloads including roleInheritance.
  • Refresh package-lock.json (lockfile metadata changes).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
package-lock.json Lockfile metadata updated (peer flags/entries adjusted).
lib/management/tenant.ts Adds roleInheritance parameter and includes it in request bodies.
lib/management/tenant.test.ts Updates/extends tests to assert roleInheritance in request payloads.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@aviadl aviadl merged commit ae4d906 into main Jan 28, 2026
33 checks passed
@aviadl aviadl deleted the tenant-inheritance branch January 28, 2026 05:57
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