Skip to content

Handle failures when opening a saved recording - #330

Merged
TobiasRoeddiger merged 1 commit into
automation/bump-version-1.5.4-36858743484from
codex/recording-open-error
Oct 5, 2026
Merged

TobiasRoeddiger merged 1 commit into
automation/bump-version-1.5.4-36858743484from
codex/recording-open-error

Conversation

@TobiasRoeddiger

Copy link
Copy Markdown
Member

When no installed app can open a saved recording, tapping it currently throws an unhandled exception and appears to do nothing. Catch the failure in both recording views and show a message pointing to Share file.

Validated on the Pixel 9a: reproduced the missing CSV viewer failure before the patch; both views now show a dismissible message, and sharing still opens and cancels normally. All 127 app tests, analysis and the Android release build pass.

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Visit the preview URL for this PR (updated for commit 1f37a33):

https://open-earable-web--pr330-codex-recording-open-x7tjondb.web.app

(expires Mon, 02 Nov 2026 05:09:37 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c7397c11177c71d8d81172cea9365829823fb41c

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

PR Build Artifacts

Full workflow run: https://github.com/OpenEarable/app/actions/runs/37266290828

@TobiasRoeddiger
TobiasRoeddiger merged commit 2379123 into automation/bump-version-1.5.4-36858743484 Oct 5, 2026
9 checks passed
@TobiasRoeddiger
TobiasRoeddiger deleted the codex/recording-open-error branch October 5, 2026 05:14
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.

1 participant