feat: Privacy policy and terms of service#13
Conversation
Signed-off-by: Dave Skender <8432125+DaveSkender@users.noreply.github.com>
Signed-off-by: Dave Skender <8432125+DaveSkender@users.noreply.github.com>
Signed-off-by: Dave Skender <8432125+DaveSkender@users.noreply.github.com>
Deploying facioquo with
|
| Latest commit: |
f469eb3
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://c7d723a9.facioquo-com.pages.dev |
| Branch Preview URL: | https://privacy.facioquo-com.pages.dev |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Dave Skender <8432125+DaveSkender@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: DaveSkender <8432125+DaveSkender@users.noreply.github.com>
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Warning Rate limit exceeded@DaveSkender has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 11 minutes and 34 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (5)
WalkthroughThis update introduces a new GitHub Actions workflow for Copilot environment setup, adds a markdownlint configuration file, and implements privacy and terms pages for the website. The footer and base styles are updated to include and style legal links, ensuring clear access to privacy and terms information. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Website
participant LegalPages
User->>Website: Visits any page
Website-->>User: Renders footer with legal links
User->>Website: Clicks "Privacy Policy" or "Terms of Service"
Website->>LegalPages: Loads privacy.md or terms.md
LegalPages-->>User: Displays legal content
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 4
♻️ Duplicate comments (1)
pages/terms.md (1)
1-11: Mirror title-case fix for Terms pageFollow the privacy page adjustment:
-title: Terms of service +title: Terms of Service … -# Terms of service +# Terms of ServiceConsistency helps search snippets and reduces duplicate‐content flags.
🧹 Nitpick comments (2)
.markdownlint.json (1)
1-6: Global rule disables may hide real issuesTurning off MD013, 25, 36, 41 project-wide is convenient but guts important readability linting (e.g., multiple H1s). Consider:
- Scoping exceptions to
pages/privacy.md,pages/terms.mdviamd003/style, or- Using
<!-- markdownlint-disable -->blocks inside the legal docs.Keeps the rest of the repo benefiting from the defaults.
pages/privacy.md (1)
1-11: Inconsistent title casing harms SEO & brandingFront-matter
title:and H1 use “Privacy policy” (lower-case “p”). Elsewhere the document refers to “Privacy Policy”. Align for consistency:-title: Privacy policy +title: Privacy Policy … -# Privacy policy +# Privacy PolicySame tweak for the Terms page to keep navigation uniform.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
Gemfile.lockis excluded by!**/*.lock
📒 Files selected for processing (6)
.github/workflows/copilot-setup-steps.yml(1 hunks).markdownlint.json(1 hunks)_includes/footer.html(1 hunks)_includes/head-base-styles.html(1 hunks)pages/privacy.md(1 hunks)pages/terms.md(1 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). (2)
- GitHub Check: Codacy Static Code Analysis
- GitHub Check: Cloudflare Pages
…o organization contact information
Privacy Policy and Terms of Service
This PR adds comprehensive Privacy Policy and Terms of Service pages to establish proper legal framework and user protections for FacioQuo services.
What's Added
/privacy) - Conservative privacy policy covering data collection, usage, and user rights/terms) - Comprehensive terms covering service usage, limitations, and legal protectionsKey Features
Privacy Policy includes:
Terms of Service includes:
Implementation Details
Legal Considerations
These documents provide broad protection while maintaining user-friendly language. They're designed to cover current and future FacioQuo services without requiring immediate customization, and are aligned to standard industry best practices.
Summary by CodeRabbit
New Features
Style
Chores