Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 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 |
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
Code Review Completed! 🔥The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Providing Context (Files & MCPs)Add these hints in your PR description (or a comment) to unlock deeper checks:
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
|
|
Preview deployed: https://pr-59.siloserver-org.pages.dev
The alias updates after a successful build and deployment. The preview is removed when this pull request closes. |
Problem
Closes #47
Silo-Server/silo-server#1672 added an Advisory Age poster badge in the web app. The profiles guide still said advisory ages appear only on title pages.
Pages changed
What changed on each page
Profiles, PINs, and shared devices
13+, in the bottom-right corner, and titles without an advisory age show no badge.Left out on purpose
Evidence
Source reviewed, not tested on a live server or device:
silo-server
17eced443(includes #1672, merge19e81846):web/src/lib/overlays/registry/ratings.ts,web/src/lib/overlays/ui-constants.ts,web/src/pages/settings/CardOverlaySettings.tsx,web/src/pages/SettingsLayout.tsx,web/src/pages/settings/PlaybackSettings.tsx.Needs a hands-on check: Nothing.
bun run test:docs: pass 8, fail 0bun run build: 98 pages built, all internal links validgit diff --check: cleanRisks
None identified. Documentation only.
Checklist
AI Disclosure
Note
Document the Advisory Age card overlay badge
Renames the advisory age docs section from title-page-specific to general, and covers both display options. Explains the bottom-right
13+badge, the missing badge when no advisory age exists, and that the Playback and Card Overlays settings work independently. Notes that profile age limits apply regardless of these settings, in profiles.md.Macroscope summarized 301f6ab.