Skip to content

Subscribe to events: Remove pivot for .net #127470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from

Conversation

JoshuaLai
Copy link
Contributor

After testing this page, we discovered that this page refers to apis that are deprecated and can no longer work.

Recommendation is to delete pivot for .net and just leave the azure cli and azure portal pivots

Copy link
Contributor

@JoshuaLai : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@v-dirichards
Copy link
Contributor

v-dirichards commented Aug 5, 2025

@pgrandhi

Can you review the proposed changes?

If you approve the new content in this PR, the commits must be moved to the private repository for automated checks and publishing. After you move the commits, close this PR (#please-close). If you need help moving the commits, contact the publicprs alias.

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Aug 5, 2025
Copy link
Contributor

Learn Build status updates of commit 147c4c5:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/communication-services/quickstarts/events/subscribe-to-events.md ⚠️Warning Details

articles/communication-services/quickstarts/events/subscribe-to-events.md

  • Line 0, Column 0: [Warning: pivot-id-unused] Pivot ID 'platform-net' is defined in 'acs-plat-azp-azcli-net-ps' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Copy link
Contributor

Learn Build status updates of commit 8919356:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/communication-services/quickstarts/events/subscribe-to-events.md ⚠️Warning Details
articles/communication-services/quickstarts/events/includes/create-event-subscription-net.md ✅Succeeded n/a (file deleted or renamed)

articles/communication-services/quickstarts/events/subscribe-to-events.md

  • Line 0, Column 0: [Warning: pivot-id-unused] Pivot ID 'platform-net' is defined in 'acs-plat-azp-azcli-net-ps' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@JoshuaLai
Copy link
Contributor Author

@v-dirichards I updated the author to @awang119 as she is the new owner for Chat. Can we update the assignee to her?

Copy link
Contributor

Learn Build status updates of commit e61bcec:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/communication-services/quickstarts/events/subscribe-to-events.md ⚠️Warning Details
articles/communication-services/quickstarts/events/includes/create-event-subscription-net.md ✅Succeeded n/a (file deleted or renamed)

articles/communication-services/quickstarts/events/subscribe-to-events.md

  • Line 0, Column 0: [Warning: pivot-id-unused] Pivot ID 'platform-net' is defined in 'acs-plat-azp-azcli-net-ps' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@v-dirichards
Copy link
Contributor

@awang119 If you approve the new content in this PR, the commits must be moved to the private repository for automated checks and publishing. After you move the commits, close this PR (#please-close). If you need help moving the commits, contact the publicprs alias.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Copy link
Contributor

Learn Build status updates of commit bc85578:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/communication-services/quickstarts/events/subscribe-to-events.md ⚠️Warning Details
articles/communication-services/quickstarts/events/includes/create-event-subscription-net.md ✅Succeeded n/a (file deleted or renamed)

articles/communication-services/quickstarts/events/subscribe-to-events.md

  • Line 0, Column 0: [Warning: pivot-id-unused] Pivot ID 'platform-net' is defined in 'acs-plat-azp-azcli-net-ps' group in 'articles/zone-pivot-groups.yml', but not used. You must use all pivot IDs in a pivot group to avoid blank tabs.

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@JoshuaLai
Copy link
Contributor Author

#please-close

@JoshuaLai JoshuaLai deleted the patch-7 branch August 11, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants