Conversation
WalkthroughImported Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/routes/docs/products/avatars/screenshots/`+page.markdoc:
- Line 257: Replace the incorrect enum name Timezone.EuropeParisAppwrite with
the correct Timezone.EuropeParis on the line that currently sets timezone:
Timezone.EuropeParisAppwrite; update that single occurrence so it matches the
other PascalCase timezone enums (e.g., Timezone.EuropeLondon,
Timezone.AsiaTokyo) and preserves the existing casing and punctuation.
🧹 Nitpick comments (2)
src/routes/docs/products/avatars/screenshots/+page.markdoc (2)
106-106: Inconsistent boolean parameter documentation.The
fullpageandtouchparameters are described using1/0values, but the code examples throughout the document usetrue/false(e.g.,fullpage: trueon line 150,touch: trueon line 269). Update the descriptions to usetrue/falsefor consistency with the examples.📝 Suggested fix
-| fullpage | boolean | Capture the full scrollable page (`1`) or only the viewport (`0`). Defaults to `false` if not provided. | +| fullpage | boolean | Capture the full scrollable page (`true`) or only the viewport (`false`). Defaults to `false` if not provided. |-| touch | boolean | Enable touch device support (`1`) or disable it (`0`). Defaults to `false` if not provided. | +| touch | boolean | Enable touch device support (`true`) or disable it (`false`). Defaults to `false` if not provided. |Also applies to: 112-112
124-237: Missing React Native example in viewport customization section.The initial code example (lines 76-90) includes
client-react-native, but this section omits it. Consider adding React Native examples for consistency, or document that subsequent examples focus on a subset of SDKs.
Co-authored-by: Atharva Deosthale <atharva.deosthale17@gmail.com>
Co-authored-by: Atharva Deosthale <atharva.deosthale17@gmail.com>
Co-authored-by: Atharva Deosthale <atharva.deosthale17@gmail.com>
Co-authored-by: Atharva Deosthale <atharva.deosthale17@gmail.com>
Co-authored-by: Atharva Deosthale <atharva.deosthale17@gmail.com>
Co-authored-by: Atharva Deosthale <atharva.deosthale17@gmail.com>
What does this PR do?
Add docs page for Screenshots API
Test Plan
Visit
/docs/products/avatars/screenshotsRelated PRs and Issues
appwrite/appwrite#10675
Have you read the Contributing Guidelines on issues?
Yes
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.