Skip to content

32437 - Update simple account flow creation email Invitation#4087

Merged
Rajandeep98 merged 3 commits intobcgov:mainfrom
Rajandeep98:32437_Unaffiliated_Email_Flow
Feb 19, 2026
Merged

32437 - Update simple account flow creation email Invitation#4087
Rajandeep98 merged 3 commits intobcgov:mainfrom
Rajandeep98:32437_Unaffiliated_Email_Flow

Conversation

@Rajandeep98
Copy link
Collaborator

@Rajandeep98 Rajandeep98 commented Feb 18, 2026

Issue #: /bcgov/entity#32437

Description of changes:

  • process update of email
  • send email if email exists and config SEND_UNAFFILIATED_EMAIL is On
  • skip sending email if config SEND_UNAFFILIATED_EMAIL is False
  • throw error if email is null and config SEND_UNAFFILIATED_EMAIL is True
  • throw error if config SEND_UNAFFILIATED_EMAIL is True and config AFFILIATE_ENTITY is True

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the lear license (Apache 2.0).

@Rajandeep98 Rajandeep98 marked this pull request as ready for review February 18, 2026 22:25
@Rajandeep98 Rajandeep98 self-assigned this Feb 18, 2026
Copy link
Collaborator

@argush3 argush3 left a comment

Choose a reason for hiding this comment

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

Pre-approving with assumption the one comment is addressed.


USE_CUSTOM_CONTACT_EMAIL=False
CUSTOM_CONTACT_EMAIL=
SEND_UNAFFILIATED_EMAIL=False
Copy link
Collaborator

Choose a reason for hiding this comment

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

thanks for adding this!

@Rajandeep98 Rajandeep98 merged commit e85a71d into bcgov:main Feb 19, 2026
1 check passed
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

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.

4 participants

Comments