Skip to content

fix(provisioning_api): read newUser.sendEmail as a boolean - #62776

Merged
susnux merged 1 commit into
masterfrom
fix/59329-welcome-email-config-type
Aug 3, 2026
Merged

fix(provisioning_api): read newUser.sendEmail as a boolean#62776
susnux merged 1 commit into
masterfrom
fix/59329-welcome-email-config-type

Conversation

@pringelmann

@pringelmann pringelmann commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

newUser.sendEmail is written as a typed boolean since #57079, but both user creation paths still compared it to the string yes

TODO

  • Backport to stable34 and stable33

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI (tests)

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
@pringelmann pringelmann added this to the Nextcloud 35 milestone Aug 3, 2026
@pringelmann pringelmann self-assigned this Aug 3, 2026
@pringelmann
pringelmann requested a review from a team as a code owner August 3, 2026 07:26
@pringelmann pringelmann added the bug label Aug 3, 2026
@pringelmann
pringelmann requested review from leftybournes, nfebe and salmart-dev and removed request for a team August 3, 2026 07:26
@pringelmann pringelmann added the 3. to review Waiting for reviews label Aug 3, 2026
@pringelmann
pringelmann requested a review from sorbaugh August 3, 2026 07:26
@pringelmann pringelmann added the team: IDP Code maintained by 🪪 IDP team label Aug 3, 2026
@susnux
susnux merged commit f707400 into master Aug 3, 2026
262 of 264 checks passed
@susnux
susnux deleted the fix/59329-welcome-email-config-type branch August 3, 2026 10:55
@pringelmann

Copy link
Copy Markdown
Contributor Author

/backport to stable34

@pringelmann

Copy link
Copy Markdown
Contributor Author

/backport to stable33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug feature: users and groups regression team: IDP Code maintained by 🪪 IDP team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: New user does not receive welcome email

3 participants