Skip to content

Conversation

@sabbene
Copy link

@sabbene sabbene commented Jan 27, 2026

Recently installed ImmichFrame and ran into a few issues not captured in the Install_Client.md docs.

  • Include link to ImmichFrame repo's release area
  • Suggest enabling "transfer from computer" setting on frameo devices to enable adb sideloading.

Summary by CodeRabbit

  • Documentation
    • Updated Android sideload instructions to point to the specific ImmichFrame_Android release location for easier access.
    • Expanded Frameo setup with a required "Transfer from computer" prerequisite and clarified setup flow.
    • Improved ADB/Android screensaver guidance and reorganized command references for clearer step-by-step use.

✏️ Tip: You can customize this high-level summary in your review settings.

Updated the Android installation instructions to include a direct link to the APK releases.
@coderabbitai
Copy link

coderabbitai bot commented Jan 27, 2026

📝 Walkthrough

Walkthrough

Documentation for client installation updated: Android APK sideload reference now points to ImmichFrame_Android Releases; Frameo instructions add an explicit "Transfer from computer" prerequisite and expand ADB usage steps; ADB reference formatting/placement adjusted.

Changes

Cohort / File(s) Summary
Documentation Updates
Install_Client.md
Android APK sideload reference changed to explicit ImmichFrame_Android Releases URL; Frameo steps now include prerequisite to enable "Transfer from computer" and expanded ADB setup flow; ADB reference formatting/placement adjusted

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

documentation

Suggested reviewers

  • jfly

Poem

🐰 I hopped through docs with glee,
Cleared the Android path for thee,
Told Frameo how to freely share,
ADB steps now tidy and fair,
🥕✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title contains a typo ('instuctions' instead of 'instructions') and is related to the changeset, but the typo makes it less professional and could be clearer about which instructions are being updated. Correct the typo: 'Update android/frameo instructions'. Consider if the title could be more specific about the core changes, such as mentioning the transfer prerequisite or APK sideload updates.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@Install_Client.md`:
- Line 89: Fix the typo "mucst" → "must" and simplify the long sentence in the
ADB/transfer instruction line: split it into shorter, clearer sentences (e.g.,
one sentence about enabling ADB via Frameo Settings → About → Enable Beta
Program and toggling ADB Access On-Off-On; a separate sentence stating that
"Transfer from computer must be enabled (Frame → Settings → Manage photos →
Transfer from computer → toggle 'Enable transfer from computer' ON)"); also
remove the duplicated "to to" so it reads "to set ImmichFrame as default Home
app." Ensure wording is concise and readable.

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