-
Notifications
You must be signed in to change notification settings - Fork 75
Update android/frameo instuctions #575
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
base: main
Are you sure you want to change the base?
Conversation
Updated the Android installation instructions to include a direct link to the APK releases.
📝 WalkthroughWalkthroughDocumentation 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
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ 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. Comment |
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.
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.
Recently installed ImmichFrame and ran into a few issues not captured in the Install_Client.md docs.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.