Repository navigation
docs: update collections and Home rows pages for the revamp - #122
Conversation
…file sharing Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Draft until Silo-Server/silo-server#2169 is merged into main. Silo-Server/silo-server#2013 has merged. The docs describe both changes, and #2169's labels may still change in review. |
Silo Kody — review completeReview finished. Check the inline comments for findings and verify each suggestion against the code and tests. Reviewing changes in Silo
Review settingsReview OptionsThe following review options are enabled or disabled:
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to The documentation is mergeable on the available evidence; check the Home-row labels against 🚥 Pre-merge checks | ✅ 5
✨ Finishing Touches
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 |
|
Preview removed after this pull request was closed. Its address now redirects to siloserver.org, though some visitors may still see the old preview for a while. |
Rows such as Critically Acclaimed now show only titles enough people rated on TMDB (Silo-Server/silo-server#2169). Say where the counts come from, how to fill them at once, and that the row preview says what an empty row looks for. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Silo Kody — review completeReview finished. Check the inline comments for findings and verify each suggestion against the code and tests. Reviewing changes in Silo
Review settingsReview OptionsThe following review options are enabled or disabled:
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fabe0f1ce3
ℹ️ 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".
| --- | ||
|
|
||
| Shared collections belong to a library and appear for everyone who can use it, for example a set of short films or a seasonal selection. | ||
| Server collections belong to one or more libraries and appear for everyone who can use them, on each library's **Collections** tab and in Home rows, for example a set of short films or a seasonal selection. |
There was a problem hiding this comment.
Describe Home-row placement as optional
A newly created server collection does not automatically appear in a Home row; the guide itself later requires the administrator to use Add as a row under Rows that show it. Saying collections appear “in Home rows” misleads administrators into expecting creation alone to place them there. Clarify that collections can be shown in optional Home or library-page rows.
Useful? React with 👍 / 👎.
Silo Kody — review completeReview finished. Check the inline comments for findings and verify each suggestion against the code and tests. Reviewing changes in Silo
Review settingsReview OptionsThe following review options are enabled or disabled:
|
…tion The app has the create screen, but nothing on iPhone or iPad opens it, and it can't add titles to a collection.
Silo Kody — review completeReview finished. Check the inline comments for findings and verify each suggestion against the code and tests. Reviewing changes in Silo
Review settingsReview OptionsThe following review options are enabled or disabled:
|
|
@Quick104 iPhone and iPad users can't make a collection yet, even on silo-apple main (checked at
The page now says the iPhone, iPad, and Android apps can open collections but can't create them, and points to the web app. We can add the steps back once the app gets a way into the screen and an add-titles action. |
Problem
Closes #121
Silo-Server/silo-server#2013 reworks collections and Home rows in the web app. The manual's collection and Home row pages describe screens, switches, and steps that change or go away when it merges: New Collection with Share with this account, Browse Templates and template bundles, Admin > Sections with Add from Gallery, and Editing scope in Settings > Home Screen.
Silo-Server/silo-server#2169 changes the rating-led rows admins add from the gallery, such as Critically Acclaimed and the mood rows: they now show only titles enough people rated on TMDB, using vote counts from the TMDB plugin 1.2.25 or later, so they can be short or empty until titles refresh.
silo-server#2013 has merged. This stays a draft until silo-server#2169 merges.
Pages changed
What changed on each page
Browse and organize collections
Home and Calendar: Change your home rows
Create shared and smart collections
Worth checking: the page title still says "Create shared and smart collections". It still fits, but "Create server collections" would match the app's wording if you prefer it.
Curate the home screen
Left out on purpose
Evidence
Source reviewed, not tested here on a live server or device: silo-server#2013 at
839f62ba1(web labels and copy inweb/src/lib/collections/copy.tsand the collection, Home rows, and Home Screen components). The labels were also seen on a sandbox walk of that PR's build during its review.Source reviewed, not tested here: silo-server#2169 at
13795af52(the vote minimums ininternal/sections/recipes, the preview's Looks for line inweb/src/components/homeRows/addRow/RowPreview.tsx, and Refresh All Metadata inweb/src/components/admin/libraries/LibraryRefreshDialog.tsxon main).Needs a hands-on check: the final labels after silo-server#2169 merges.
Follow-ups
Risks
None identified. Documentation only.
Checklist
AI Disclosure
Note
Update collections and Home rows docs for the collections revamp
Macroscope summarized fe6ebd1.