Skip to content

[WIP] Update null TenantId implementation for tenant-agnostic entities#7228

Closed
Copilot wants to merge 1 commit intoenh/tenant-agnostic-entitiesfrom
copilot/sub-pr-7226-again
Closed

[WIP] Update null TenantId implementation for tenant-agnostic entities#7228
Copilot wants to merge 1 commit intoenh/tenant-agnostic-entitiesfrom
copilot/sub-pr-7226-again

Conversation

Copy link
Contributor

Copilot AI commented Jan 31, 2026

Thanks for the feedback on #7226. I've created this new PR, which merges into #7226, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #7226
Triggering review: #7226 (comment)

@copilot open a new pull request to apply changes based on this feedback

However, be careful that we still remove stale descriptors when eg. workflow definitions are removed, after which we refresh the registry of descriptors, which may no longer "see" the delete workflow definition, therefore its descriptor becoming stale. There is at lest on component test that guards against this, so make sure to run them whenever you make a change here.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@sfmskywalker sfmskywalker marked this pull request as ready for review January 31, 2026 09:26
Copilot AI review requested due to automatic review settings January 31, 2026 09:26
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 wasn't able to review any files in this pull request.

@sfmskywalker sfmskywalker deleted the copilot/sub-pr-7226-again branch February 2, 2026 09: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.

3 participants