Skip to content

Installs: Cancel isn't a failure, links can replace an expired code, translated update note - #464

Merged
McCal-Codes merged 5 commits into
mainfrom
install-fixes
Oct 22, 2026
Merged

McCal-Codes merged 5 commits into
mainfrom
install-fixes

Conversation

@McCal-Codes

Copy link
Copy Markdown
Owner

Three more from the 20 Oct review.

  • Cancel on Android's install prompt is no longer treated as a failure. A software update goes back to Ready (the download is still good) and a Market install goes quiet.
  • A supporter link can replace a code that has run out. A code that still works is still never replaced by a link.
  • "Folio was updated to ..." is translated, and only appears after an update Folio installed on its own (the idle job), not one you tapped.

Checked: InstallOutcomeTest, unit tests and lint. Not tried on the phone (the cancel prompt and an expired code were not exercised).

…ce an expired code, and the updated note is translated
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-21T17:20:35.658395Z ae71f6d PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ae71f6dc6b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread app/src/main/java/com/mccal/folio/SoftwareUpdate.kt Outdated
Comment thread app/src/main/java/com/mccal/folio/SoftwareUpdate.kt Outdated
@McCal-Codes
McCal-Codes merged commit 471302f into main Oct 22, 2026
3 checks passed
@McCal-Codes
McCal-Codes deleted the install-fixes branch October 22, 2026 00:44
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