Skip to content

Conversation

@mikescamell
Copy link
Contributor

@mikescamell mikescamell commented Oct 31, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/1207908166761516/task/1211794252593377?focus=true

Description

Added support for launching DuckChat settings with native settings only. This PR introduces:

  1. A new DuckChatNativeSettingsNoParams activity parameter to launch DuckChat settings with search section hidden
  2. Updated the DuckChatSettingsViewModel to use assisted injection for configuring visibility of the search section
  3. Modified the OpenNativeSettingsHandler to use the new parameter when launching AI features settings
  4. Refactored the settings activity to properly handle both parameter types

Steps to test this PR

See Asana task

UI changes

Search section hidden when accessed from SERP

Screenshot_20251031_175815.png
Screenshot_20251031_175804.png

@mikescamell mikescamell force-pushed the feature/mike/serp-settings-sync/hide-search-section-when-launching-from-serp branch 2 times, most recently from b2898fa to b80eeeb Compare November 3, 2025 09:24
The logic for showing or hiding the "AI Chat for Search" settings section has been extracted into a new private function, `renderSearchSettingsSection`.
@mikescamell mikescamell force-pushed the feature/mike/serp-settings-sync/hide-search-section-when-launching-from-serp branch from b80eeeb to 88863d2 Compare November 3, 2025 10:04
@mikescamell mikescamell force-pushed the feature/mike/serp-settings-sync/hide-search-section-when-launching-from-serp branch from 88863d2 to 18a2c36 Compare November 3, 2025 10:48
@mikescamell mikescamell marked this pull request as ready for review November 3, 2025 11:11
Copy link
Contributor

@marcosholgado marcosholgado left a comment

Choose a reason for hiding this comment

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

LGTM

@mikescamell mikescamell merged commit 0bf5603 into develop Nov 4, 2025
10 checks passed
@mikescamell mikescamell deleted the feature/mike/serp-settings-sync/hide-search-section-when-launching-from-serp branch November 4, 2025 14:09
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