Skip to content

Fix download failure notification body text#827

Open
salmanmkc wants to merge 363 commits into
google-ai-edge:mainfrom
salmanmkc:salmanmkc/fix-failed-download-notification
Open

Fix download failure notification body text#827
salmanmkc wants to merge 363 commits into
google-ai-edge:mainfrom
salmanmkc:salmanmkc/fix-failed-download-notification

Conversation

@salmanmkc

@salmanmkc salmanmkc commented May 12, 2026

Copy link
Copy Markdown

Fixes #704.

When a model download fails, the notification has a failure title ("Model download failed") but the body uses notification_content_success ("Model %s has been downloaded") so users get told the model finished downloading when it actually failed. Swap the body string to notification_content_fail which already exists and uses the same %s placeholder.

Test plan

  • ./gradlew :app:compileDebugKotlin passes locally.
  • The reporter already pinpointed the line; behavioural test on a real device needs a forced download failure, which is hard to reproduce reliably in an emulator.

jinjingforever and others added 30 commits August 20, 2025 16:03
PiperOrigin-RevId: 798305739
PiperOrigin-RevId: 798306012
PiperOrigin-RevId: 799245337
PiperOrigin-RevId: 800107395
PiperOrigin-RevId: 800516780
PiperOrigin-RevId: 800638495
PiperOrigin-RevId: 802190892
…d allowlist" dialog dismissable.

PiperOrigin-RevId: 803496612
PiperOrigin-RevId: 805432594
- Hide keyboard after sending a prompt.
- Scroll to bottom when a new prompt is sent.
- Show warning when user tries to import a web model.

PiperOrigin-RevId: 805909987
PiperOrigin-RevId: 807856963
yishuangP and others added 29 commits May 5, 2026 13:15
PiperOrigin-RevId: 907029492
…y using deterministic short IDs.

PiperOrigin-RevId: 907072316
PiperOrigin-RevId: 907086171
PiperOrigin-RevId: 907174331
PiperOrigin-RevId: 907388189
PiperOrigin-RevId: 908801411
PiperOrigin-RevId: 910146658
- Added `SystemPromptRepository` to persist custom prompts via DataStore.
- Added `SystemPromptHelper` to resolve the effective prompt (custom override vs. default fallback)

PiperOrigin-RevId: 910346906
PiperOrigin-RevId: 910995322
PiperOrigin-RevId: 911749453
…iongemma-typo

fix: correct FunctionGemma typo in README
…mini app experience.

Key Changes:
- **Smoother Token Streaming**: Replaced the token-by-token display with a fade-in effect that reveals buffered chunks of tokens during model response streaming.
- **Updated Scrolling Behavior**: Disabled auto-scrolling. Introduced a "Jump to bottom" button that appears whenever the content area is scrollable.
- **Cleaner Model Response**: Remove the "bubble" around the model response text and make text full width.
- **Optimized Screen Space (Chat Only)**: When a new user prompt is sent, the view now automatically scrolls to align the prompt with the top edge of the screen, maximizing the visible area for the incoming response.

PiperOrigin-RevId: 912129993
…_current_date_and_time" intents.

PiperOrigin-RevId: 912136923
PiperOrigin-RevId: 913067368
PiperOrigin-RevId: 913755747
…hat. Introduces a new `schedule_notification` action in IntentHandler.

PiperOrigin-RevId: 913782888
…fications in Agent Chat.

PiperOrigin-RevId: 913888328
…cations scheduled via the "schedule-notification" skill to open the LLM Agent Chat screen with a pre-filled query.

PiperOrigin-RevId: 913949881
The failure branch in DownloadRepository was formatting the notification
body with notification_content_success ("Model X has been downloaded")
while showing the failure title, so users got a contradictory message
saying the model finished downloading when it actually failed. Use
notification_content_fail instead.
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.

[BUG]