Skip to content

Conversation

davidortinau
Copy link
Contributor

@davidortinau davidortinau commented Aug 19, 2025

…View/CarouselView on iOS/Mac Catalyst in .NET 10; update ms.date
@davidortinau davidortinau requested a review from a team as a code owner August 19, 2025 13:09
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.

Copy link

Learn Build status updates of commit 0096be9:

✅ Validation status: passed

File Status Preview URL Details
docs/user-interface/controls/carouselview/index.md ✅Succeeded View (>=net-maui-8.0)
docs/user-interface/controls/collectionview/index.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

Copy link
Member

@jfversluis jfversluis left a comment

Choose a reason for hiding this comment

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

I do think we want to add guidance here on how to opt-out?

Added instructions to revert to .NET 9 behavior for CarouselView.
Added instructions to revert to .NET 9 behavior for CarouselView in MauiProgram.cs.
Updated the recommendation to use the new handler for CollectionView instead of CarouselView.
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.

Copy link

Learn Build status updates of commit af0b663:

✅ Validation status: passed

File Status Preview URL Details
docs/user-interface/controls/carouselview/index.md ✅Succeeded View (>=net-maui-8.0)
docs/user-interface/controls/collectionview/index.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

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.

Copy link

Learn Build status updates of commit e06352e:

✅ Validation status: passed

File Status Preview URL Details
docs/user-interface/controls/carouselview/index.md ✅Succeeded View (>=net-maui-8.0)
docs/user-interface/controls/collectionview/index.md ✅Succeeded View (>=net-maui-8.0)

For more details, please refer to the build report.

@jfversluis jfversluis merged commit 9984368 into main Aug 27, 2025
5 checks passed
@jfversluis jfversluis deleted the pr12-cv-handler-defaults-net10-from-main branch August 27, 2025 08:39
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