Skip to content

Update cpp-console.cpp - #238

Merged
Lauren Ciha (lauren-ciha) merged 1 commit into
microsoft:mainfrom
drewbatgit:patch-1
Oct 30, 2025
Merged

Lauren Ciha (lauren-ciha) merged 1 commit into
microsoft:mainfrom
drewbatgit:patch-1

Conversation

@drewbatgit

Copy link
Copy Markdown
Contributor

Description

This change is just updating to occurrences of "Azure AppId" to "Azure ObjectId" in comments. This reflects changes that have been made on the Azure service side for Push Notifications.

Target Release

1.2 Preview 1

Checklist

Since these are minor changes to comments only, I'm hoping that this sample checklist is unnecessary in this case.

  • Samples build and run using the Visual Studio versions listed in the Windows development docs.
  • Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
  • Samples set the minimum supported OS version to Windows 10 version 1809.
  • Samples build clean with no warnings or errors.
  • [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
  • If I am onboarding a new feature, then I must have correctly setup a new CI pipeline for my feature with the correct triggers and path filters laid out in the "Onboarding Samples CI Pipeline for new feature" section in samples-guidelines.md.
  • I have commented on my PR /azp run SamplesCI-<FeatureName> to have the CI build run on my branch for each of my FeatureName my PR is modifying. This must be done on the latest commit on the PR before merging to ensure the build is up to date and accurate. Warning: the PR will not block automatically if this is not run due to '/azp run' limitation on triggering more than 10 pipelines.

@btueffers

Copy link
Copy Markdown
Contributor

/azp run SamplesCI-Notifications

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@lauren-ciha

Copy link
Copy Markdown
Member

Resolves #305

@lauren-ciha

Lauren Ciha (lauren-ciha) commented Aug 25, 2025 •

Copy link
Copy Markdown
Member

/azp run

@lauren-ciha
Lauren Ciha (lauren-ciha) merged commit 70e13b7 into microsoft:main Oct 30, 2025
6 checks passed
shisan (qiutongMS) pushed a commit that referenced this pull request Aug 25, 2026
Port the AppId-to-ObjectId documentation correction from main PR #238.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

3 participants