Skip to content

Keep disabled subscriptions out of confirmation emails - #169

Draft
yvzn with Copilot wants to merge 2 commits into
mainfrom
copilot/show-disabled-subscriptions
Draft

yvzn with Copilot wants to merge 2 commits into
mainfrom
copilot/show-disabled-subscriptions

Conversation

Copilot AI commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

The Check Subscription page showed matching locations without indicating which were disabled, and confirmations could include disabled subscriptions. All-disabled matches could therefore appear to be active subscriptions.

  • Admin results: Keep all matching locations visible, label disabled entries, and show a warning when none are active.
  • Email preview and sending: Use active locations only for confirmation content and app links; block sending when no active locations remain.
mailTemplates.For(SelectedRequestLang).SubscriptionConfirmation(
    ActiveLocations,
    ContactEmail,
    addLinkToApp: isAppRequest);

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: yvzn <19163871+yvzn@users.noreply.github.com>
Copilot AI changed the title [WIP] Update Check Subscription to show disabled subscriptions Keep disabled subscriptions out of confirmation emails Nov 13, 2026
Copilot AI requested a review from yvzn November 13, 2026 08:02

This branch has not been deployed

No deployments
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.

Show disabled subscriptions in Check Subscription but exclude them from confirmation emails

2 participants