Skip to content

chore: fix privacy policy table formatting and alignment #2654

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.

Sign up for GitHub

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

Merged
stnguyen90 merged 2 commits into from
Dec 12, 2025
Merged

chore: fix privacy policy table formatting and alignment #2654

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
882befd
chore: fix privacy policy table formatting and alignment
stnguyen90 Dec 10, 2025
3a9b84e
Merge branch 'main' into fix-privacy-tables
DylanG-64 Dec 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 22 additions & 21 deletions src/routes/privacy/+page.markdoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,12 @@ What personal data we collect, why we collect it, and how it is used
{% table %}
---
* Specific personal data we collect
* Address
* Full name
* Email address
* IP address
* Country
* Payment details
* - Address
- Full name
- Email address
- IP address
- Country
- Payment details
---
* The purpose of data collection
* For payment processing purposes
Expand All @@ -49,8 +49,8 @@ What personal data we collect, why we collect it, and how it is used
- Legitimate interest
---
* Third parties with whom we share your personal data
* Stripe
* DigitalOcean
* - Stripe
- DigitalOcean
{% /table %}


Expand All @@ -68,42 +68,43 @@ What personal data we collect, why we collect it, and how it is used
- IP address
- Country
- User-agent (info about the browser)
* - Phone number
* - Usage details
- Phone number
- Usage details
* - Full name
- Email address
Comment on lines +71 to +74
Copy link
Contributor

@coderabbitai coderabbitai bot Dec 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

cd /repo && wc -l src/routes/privacy/+page.markdoc

Repository: appwrite/website

Length of output: 115

🏁 Script executed:

cd /repo && sed -n '60,110p' src/routes/privacy/+page.markdoc

Repository: appwrite/website

Length of output: 115

🏁 Script executed:

pwd && ls -la

Repository: appwrite/website

Length of output: 2369

🏁 Script executed:

fd "+page.markdoc" | head -20

Repository: appwrite/website

Length of output: 352

🏁 Script executed:

sed -n '60,110p' src/routes/privacy/+page.markdoc

Repository: 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.

---
* The purpose of data collection
* - To be able to create an account
- To be able to log in
- To be able to enjoy features available to registered users
- To collect Informative reason for the administrator
- To identify reported problems during project development
* - To be able to log in, using two factor authentication via text message
* - To be able to monitor errors
- To be able to log in, using two factor authentication via text message
- To be able to monitor errors
* - To send you marketing communications and product updates
---
* Legal basis (GDPR only, if applicable)
* - Processing is necessary for the performance of a contract to which the data subject is party or in order to take steps at the request of the data subject prior to entering into a contract
- Legitimate interest (e.g. to allow you to create an account and log-in)
* Consent
* Legitimate interest
* Consent
* - Consent
---
* Third parties with whom we share your personal data
- Userlist
* - MSG91
* - Userlist
- MSG91
- Activetrail (Israel only)
- SMS.to
* - Sentry.io
- Sentry.io
* - Userlist
Comment on lines +92 to 97
Copy link
Contributor

@coderabbitai coderabbitai bot Dec 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 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.

Suggested change
* - 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.

---
* Consequences of not providing the Personal Data
* - Cannot create an account
- Cannot log in
- Cannot enjoy features available to registered users
- Cannot identify reported problem during your project development process
* - Cannot log in using two factor authentication based on phone number
* - Cannot monitor and solve errors
* - Cannot sign up using our website forms
- Cannot log in using two factor authentication based on phone number
- Cannot monitor and solve errors
- Cannot sign up using our website forms
*
Copy link
Contributor

@coderabbitai coderabbitai bot Dec 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

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.

Suggested change
*
🤖 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.

{% /table %}

#### When you purchase products in our Swag Store
Expand Down