chore: fix privacy policy table formatting and alignment #2654
Conversation
This PR fixes the tables to put the contents in the right cells.
WalkthroughThe changes reformat bullet list entries in the privacy policy markdown documentation. Multiple sections including "Specific personal data we collect," "Third parties with whom we share your personal data," "Consent/Legal basis," and "Consequences of not providing the personal data" are restructured from flat lists to nested sub-bullet lists. All changes are purely presentational within table formatting, with no modifications to logic, control flow, or semantic content. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/routes/privacy/+page.markdoc(3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: assets
- GitHub Check: build
- GitHub Check: tests
🔇 Additional comments (2)
src/routes/privacy/+page.markdoc (2)
37-42: LGTM — Nested bullet formatting is clean and consistent.
52-53: LGTM — Consistent nested list formatting.
| - Phone number | ||
| - Usage details | ||
| * - Full name | ||
| - Email address |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
cd /repo && wc -l src/routes/privacy/+page.markdocRepository: appwrite/website
Length of output: 115
🏁 Script executed:
cd /repo && sed -n '60,110p' src/routes/privacy/+page.markdocRepository: appwrite/website
Length of output: 115
🏁 Script executed:
pwd && ls -laRepository: appwrite/website
Length of output: 2369
🏁 Script executed:
fd "+page.markdoc" | head -20Repository: appwrite/website
Length of output: 352
🏁 Script executed:
sed -n '60,110p' src/routes/privacy/+page.markdocRepository: appwrite/website
Length of output: 1712
🏁 Script executed:
cat -n src/routes/privacy/+page.markdoc | sed -n '64,107p'Repository: appwrite/website
Length of output: 1898
Remove duplicate entries in the "Specific personal data we collect" section.
Lines 71–74 duplicate data already in the preceding cell:
- Line 71 repeats "Phone number" from line 67
- Lines 73–74 repeat "Full name" and "Email address" from lines 65–66
Clarify whether "Usage details" (line 72) is intentional new content or was mistakenly added during restructuring. If intentional, confirm its placement; if not, remove it along with the other duplicates.
🤖 Prompt for AI Agents
In src/routes/privacy/+page.markdoc around lines 71–74, remove the duplicate
list items ("Phone number", "Full name", "Email address") that repeat entries
already present earlier, and either delete "Usage details" if it was added
accidentally or, if it is intentional, confirm its intent and move it into the
correct section with a brief clarifying label (e.g., "Usage details (e.g., app
usage, feature interactions)") so the list has no duplicates and each item is
clearly placed.
| * - Userlist | ||
| - MSG91 | ||
| - Activetrail (Israel only) | ||
| - SMS.to | ||
| * - Sentry.io | ||
| - Sentry.io | ||
| * - Userlist |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, check if the file exists and get its content around lines 92-97
cat -n src/routes/privacy/+page.markdoc | sed -n '85,105p'Repository: appwrite/website
Length of output: 1025
Remove duplicate "Userlist" entry on line 97.
"Userlist" appears as a separate entry on line 97 with no nested items, duplicating the entry on line 92. Remove line 97 only, retaining all nested service providers (MSG91, Activetrail, SMS.to, Sentry.io) under the first Userlist entry.
- SMS.to
- Sentry.io
-* - Userlist📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| * - Userlist | |
| - MSG91 | |
| - Activetrail (Israel only) | |
| - SMS.to | |
| * - Sentry.io | |
| - Sentry.io | |
| * - Userlist | |
| * - Userlist | |
| - MSG91 | |
| - Activetrail (Israel only) | |
| - SMS.to | |
| - Sentry.io |
🤖 Prompt for AI Agents
In src/routes/privacy/+page.markdoc around lines 92 to 97, there is a duplicate
top-level "Userlist" entry on line 97 that should be removed; keep the original
"Userlist" entry at line 92 along with its nested items (MSG91, Activetrail,
SMS.to, Sentry.io) and delete only the redundant standalone "Userlist" line so
the list is not duplicated.
| - Cannot log in using two factor authentication based on phone number | ||
| - Cannot monitor and solve errors | ||
| - Cannot sign up using our website forms | ||
| * |
There was a problem hiding this comment.
Remove the empty bullet point on line 107.
Line 107 contains * with no content, which represents an empty bullet point. This appears to be a formatting artifact from the restructuring. Verify if this is intentional or a copy-paste error, and remove if unintended.
- Cannot sign up using our website forms
-*
{% /table %}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| * |
🤖 Prompt for AI Agents
In src/routes/privacy/+page.markdoc at line 107, remove the empty bullet point
that consists solely of "* " (a lone list marker) — delete that line so there is
no stray empty list item; verify surrounding list indentation/blank lines remain
valid and run a quick markdown/markdoc render check to ensure no formatting
regressions.
What does this PR do?
This PR fixes the tables to put the contents in the right cells.
Test Plan
Manual:
Related PRs and Issues
Previous privacy PRs:
Have you read the Contributing Guidelines on issues?
Yes
Summary by CodeRabbit
Release Notes
✏️ Tip: You can customize this high-level summary in your review settings.