Skip to content

Conversation

@jfversluis
Copy link
Member

@jfversluis jfversluis commented Oct 21, 2025

Fixes a broken link


Internal previews

📄 File 🔗 Preview link
docs/data-cloud/push-notifications.md docs/data-cloud/push-notifications

@jfversluis jfversluis requested a review from a team as a code owner October 21, 2025 08:28
@learn-build-service-prod
Copy link

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link

Learn Build status updates of commit b157a9f:

💡 Validation status: suggestions

File Status Preview URL Details
docs/data-cloud/push-notifications.md 💡Suggestion View (>=net-maui-8.0) Details

docs/data-cloud/push-notifications.md

  • Line 2033, Column 76: [Suggestion: other-site-link-broken - See documentation] Link 'https://firebase.google.com/docs/cloud-messaging/concept-options' points to a page that doesn't exist. Check the path or URL and update the link.

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.

@jfversluis jfversluis enabled auto-merge (squash) October 21, 2025 20:44
@jfversluis
Copy link
Member Author

The suggestion is for the link I'm replacing somehow. The preview looks good and has the new link. Seems like a false positive.

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.

2 participants