Skip to content

Send notifications even when we're unpackaged#19937

Open
zadjii-msft wants to merge 3 commits intodev/migrie/fhl-spring26/bellStyle-notificationfrom
dev/migrie/fhl-spring26/unpackaged-notify
Open

Send notifications even when we're unpackaged#19937
zadjii-msft wants to merge 3 commits intodev/migrie/fhl-spring26/bellStyle-notificationfrom
dev/migrie/fhl-spring26/unpackaged-notify

Conversation

@zadjii-msft
Copy link
Member

Targets #19936

Manually assign an AUMID to our process when we're running unpackaged. That'll let us actually send Windows notifications when we're unpackaged.

@zadjii-msft
Copy link
Member Author

big s/o to @shanselman who had this in Toasty and I copied liberally ❤️

@github-actions

This comment has been minimized.

Comment on lines +389 to +395
LOG_IF_FAILED(SetCurrentProcessExplicitAppUserModelID(L"Microsoft.WindowsTerminal"));
#elif defined(WT_BRANDING_PREVIEW)
LOG_IF_FAILED(SetCurrentProcessExplicitAppUserModelID(L"Microsoft.WindowsTerminalPreview"));
#elif defined(WT_BRANDING_CANARY)
LOG_IF_FAILED(SetCurrentProcessExplicitAppUserModelID(L"Microsoft.WindowsTerminalCanary"));
#else
LOG_IF_FAILED(SetCurrentProcessExplicitAppUserModelID(L"Microsoft.WindowsTerminalDev"));
Copy link
Member

Choose a reason for hiding this comment

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

how does this work for different versions of portable mode? we have already (and a couple times) fixed issues where portable and unpackaged have "crosstalk". See how we generate the Window ID in the WindowEmperor.

Copy link
Member

Choose a reason for hiding this comment

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

This identifier allows an application to group its associated processes and windows under a single taskbar button.

for example, we do not want different instances of portable mode or unpackaged to glom together with packaged or with eachother.

Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

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

noted within

Comment on lines 63 to +74
@@ -70,6 +71,7 @@
"audible",
"taskbar",
"window",
"notification",
Copy link
Member

Choose a reason for hiding this comment

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

wrong PR

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Mar 5, 2026
@microsoft-github-policy-service microsoft-github-policy-service bot added the No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity. label Mar 12, 2026
@microsoft-github-policy-service
Copy link
Contributor

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 7 days. It will be closed if no further activity occurs within 7 days of this comment.

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

Labels

Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants