Greptile Summary

Added a "None" option at the top of the group host picker dialog to allow users to clear the selected location sharing host. The option is conditionally displayed only when a host is currently selected, and properly triggers the unset callback to clear the collector address.

• Added onUnset callback parameter to GroupHostPickerDialog
• Implemented conditional "None" option in the contact picker using selectedHash != null check
• Used consistent UI styling with Person icon and HorizontalDivider separator
• Properly integrated with TelemetryCollectorSection to pass null to onCollectorAddressChange

Confidence Score: 5/5

• This PR is safe to merge with no risks
• The changes are minimal, well-contained, and follow existing patterns. The conditional rendering logic is correct, the callback properly handles the unset action, and the UI styling is consistent with existing code. No logical errors or potential bugs were identified.
• No files require special attention

Important Files Changed

Flowchart

flowchart TD
    A[User Opens Group Host Picker] --> B{Is Host Selected?}
    B -->|Yes| C[Show 'None' Option]
    B -->|No| D[Show Contact List Only]
    C --> E[Display Contact List]
    D --> E
    E --> F{User Action}
    F -->|Clicks 'None'| G[onUnset Callback]
    F -->|Selects Contact| H[onContactSelected Callback]
    F -->|Dismisses Dialog| I[onDismiss Callback]
    G --> J[Set Collector Address to null]
    H --> K[Set Collector Address to Contact Hash]
    I --> L[Close Dialog]
    J --> L
    K --> L

_{Last reviewed commit: 6c75a33}

Codecov Report

❌ Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

📢 Thoughts on this report? Let us know!