Skip to content

Update delegated user plans on login#4385

Merged
stkillen merged 3 commits into
mainfrom
bugs/539137
Aug 28, 2025
Merged

Update delegated user plans on login#4385
stkillen merged 3 commits into
mainfrom
bugs/539137

Conversation

@stkillen

@stkillen stkillen commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Summary

Problem
Delegated plans are not updated on user sync. If they're ever changed, we do not update them.

Additionally, a delegated user can end up without any plans assigned, and will never get one assigned.

Solution
Update delegated user plans on login to ensure delegated users have a plan.

Work Item(s)

Fixes AB#539137

@dannoe

dannoe commented Aug 20, 2025

Copy link
Copy Markdown
Contributor

Out of curiosity: In a customer tenant, I am a delegated admin. My user account was created before the GDPR regulations came into effect (hiding the real email address, etc.). I don't have a plan attached (it's not shown on the user card, and the user plan query doesn't list me either). Does this PR fix this problem?

@stkillen

Copy link
Copy Markdown
Contributor Author

Out of curiosity: In a customer tenant, I am a delegated admin. My user account was created before the GDPR regulations came into effect (hiding the real email address, etc.). I don't have a plan attached (it's not shown on the user card, and the user plan query doesn't list me either). Does this PR fix this problem?

Yes; that's the goal of this PR. Just finishing some validation testing

@stkillen stkillen closed this Aug 26, 2025
@stkillen stkillen reopened this Aug 26, 2025
@github-actions github-actions Bot modified the milestones: Version 27.0, Version 28.0 Aug 27, 2025
@stkillen stkillen marked this pull request as ready for review August 27, 2025 11:08
@stkillen stkillen requested a review from a team as a code owner August 27, 2025 11:08
@stkillen stkillen enabled auto-merge (squash) August 27, 2025 11:08
darjoo
darjoo previously approved these changes Aug 28, 2025
Comment thread src/System Application/App/Azure AD Plan/src/AzureADPlanImpl.Codeunit.al Outdated
@stkillen stkillen merged commit d20dbe4 into main Aug 28, 2025
25 checks passed
@stkillen stkillen deleted the bugs/539137 branch August 28, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants