-
Notifications
You must be signed in to change notification settings - Fork 1.3k
CMM-837 oddie bot support UI #22284
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
CMM-837 oddie bot support UI #22284
Conversation
Generated by 🚫 Danger |
Project manifest changes for WordPressThe following changes in the --- ./build/reports/diff_manifest/WordPress/wordpressVanillaRelease/base_manifest.txt 2025-10-15 06:16:23.040524728 +0000
+++ ./build/reports/diff_manifest/WordPress/wordpressVanillaRelease/head_manifest.txt 2025-10-15 06:16:26.350572105 +0000
@@ -491,6 +491,10 @@
<activity
android:name="org.wordpress.android.ui.subscribers.SubscribersActivity"
android:label="@string/subscribers"
+ android:theme="@style/WordPress.NoActionBar" />
+ <activity
+ android:name="org.wordpress.android.support.aibot.ui.AIBotSupportActivity"
+ android:label="@string/ai_bot_conversations_title"
android:theme="@style/WordPress.NoActionBar" /> <!-- Deep Linking Activity -->
<activity
android:name="org.wordpress.android.ui.deeplinks.DeepLinkingIntentReceiverActivity" Go to https://buildkite.com/automattic/wordpress-android/builds/23622/canvas?sid=0199e67f-0d5d-440b-9d00-e9322119cc95, click on the |
Project manifest changes for WordPressThe following changes in the --- ./build/reports/diff_manifest/WordPress/jetpackVanillaRelease/base_manifest.txt 2025-10-15 06:17:46.020328045 +0000
+++ ./build/reports/diff_manifest/WordPress/jetpackVanillaRelease/head_manifest.txt 2025-10-15 06:17:50.330331251 +0000
@@ -625,6 +625,10 @@
<activity
android:name="org.wordpress.android.ui.subscribers.SubscribersActivity"
android:label="@string/subscribers"
+ android:theme="@style/WordPress.NoActionBar" />
+ <activity
+ android:name="org.wordpress.android.support.aibot.ui.AIBotSupportActivity"
+ android:label="@string/ai_bot_conversations_title"
android:theme="@style/WordPress.NoActionBar" /> <!-- Reader Activities -->
<activity
android:name="org.wordpress.android.ui.reader.ReaderPostListActivity" Go to https://buildkite.com/automattic/wordpress-android/builds/23622/canvas?sid=0199e67f-0d5e-4b26-ab31-dbaf65ad6324, click on the |
|
App Name | ![]() |
|
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr22284-167992c | |
Commit | 167992c | |
Direct Download | wordpress-prototype-build-pr22284-167992c.apk |
|
App Name | ![]() |
|
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr22284-167992c | |
Commit | 167992c | |
Direct Download | jetpack-prototype-build-pr22284-167992c.apk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces an AI bot support feature for WordPress, adding conversational UI screens for users to interact with a support bot named "Odie". The implementation uses dummy data for the initial iteration to demonstrate the UI functionality before connecting to real APIs.
- Creates a new conversational support interface with list and detail views
- Implements Jetpack Compose UI components for modern Android UI patterns
- Adds theming support and navigation between conversation screens
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 5 comments.
Show a summary per file
File | Description |
---|---|
strings.xml | Added localized strings for AI bot UI elements and time formatting |
AppThemeM3.kt | Enhanced theme functions to accept isJetpackApp parameter for flexible theming |
HelpActivity.kt | Added temporary hardcoded navigation to AI bot support screen |
ConversationUtils.kt | Implemented time formatting utilities and sample conversation data generation |
ConversationsListScreen.kt | Created conversation list UI with Material 3 design |
ConversationDetailScreen.kt | Implemented chat interface with message bubbles and input handling |
AIBotSupportViewModel.kt | Added ViewModel for managing conversation state and dummy API interactions |
AIBotSupportActivity.kt | Main activity coordinating navigation between conversation screens |
BotMessage.kt, BotConversation.kt | Data models for conversation and message entities |
AndroidManifest.xml | Registered new AI bot support activity |
build.gradle, settings.gradle | Minor formatting changes |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
WordPress/src/main/java/org/wordpress/android/support/aibot/ui/AIBotSupportViewModel.kt
Outdated
Show resolved
Hide resolved
WordPress/src/main/java/org/wordpress/android/support/aibot/ui/ConversationsListScreen.kt
Show resolved
Hide resolved
WordPress/src/main/java/org/wordpress/android/support/aibot/ui/AIBotSupportViewModel.kt
Show resolved
Hide resolved
WordPress/src/main/java/org/wordpress/android/support/aibot/ui/ConversationDetailScreen.kt
Show resolved
Hide resolved
WordPress/src/main/java/org/wordpress/android/support/aibot/util/ConversationUtils.kt
Show resolved
Hide resolved
} else { | ||
createNewZendeskTicket() | ||
} | ||
// TODO: comment hardcoded lines. This TODO is preventing the PR to pass |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignore these changes. The TODO is preventing me to merge the PR until I remove them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works great for me on-device.
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## trunk #22284 +/- ##
==========================================
- Coverage 39.85% 39.66% -0.20%
==========================================
Files 2168 2174 +6
Lines 102904 103416 +512
Branches 14840 14865 +25
==========================================
Hits 41015 41015
- Misses 58409 58921 +512
Partials 3480 3480 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Description
This PR is adding a new screen to start bot conversations with Odie. The screen is using dummy data as the real data request is being done in a second iteration for PR clarity.
Although the data is not real, you should be able to see and write conversations.
Testing instructions
Note: the entry point to this screen is hardcoded, and a TODO has been added to avoid merging this PR until those hardcoded lines are removed. So, don't pay attention to that part of the code.