Skip to content

feat(pages): add Terms of Service and Privacy Policy pages - #180

Open
prathamkhatwani wants to merge 1 commit into
AOSSIE-Org:mainfrom
prathamkhatwani:fix/footer-dead-links-and-routes
Open

feat(pages): add Terms of Service and Privacy Policy pages#180
prathamkhatwani wants to merge 1 commit into
AOSSIE-Org:mainfrom
prathamkhatwani:fix/footer-dead-links-and-routes

Conversation

@prathamkhatwani

@prathamkhatwani prathamkhatwani commented Aug 20, 2026

Copy link
Copy Markdown

Addressed Issues:

Fixes #114

📝 Description

This PR resolves the dead link issue for Terms of Service and Privacy Policy in the footer by introducing dedicated, responsive static pages and routing:

  1. Terms of Service (/terms): Outlines open source usage (MIT license), GitHub API compliance, and disclaimer of warranties.
  2. Privacy Policy (/privacy): Clarifies that OrgExplorer operates entirely client-side without backend telemetry, and explains local storage of Personal Access Tokens (PAT) and cache data.
  3. Routing: Registered /terms and /privacy in App.jsx and updated Footer.jsx so users no longer hit fallback redirects.

Screenshots/Recordings:

image image

Checklist

  • My code follows the project's code style and conventions
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have joined the Discord server and I will share a link to this PR with the project maintainers there
  • I have read the Contributing Guidelines

Summary by CodeRabbit

  • New Features
    • Added dedicated Terms of Service and Privacy Policy pages.
    • Added navigation routes for accessing both pages.
    • Added “Back to Home” navigation on policy pages.
    • Updated footer links to correctly distinguish internal pages, external websites, and email links.
    • Added hover styling for footer navigation links.

@github-actions github-actions Bot added bug Something isn't working frontend Frontend changes javascript JavaScript/TypeScript changes size/L 201-500 lines changed labels Aug 20, 2026
@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a595c85a-2fca-4424-b33c-e05797b004d0

📥 Commits

Reviewing files that changed from the base of the PR and between 2098d23 and 353007b.

📒 Files selected for processing (4)
  • src/App.jsx
  • src/components/layout/Footer.jsx
  • src/pages/PrivacyPage.jsx
  • src/pages/TermsPage.jsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The application adds Terms of Service and Privacy Policy pages, registers /terms and /privacy routes, and updates footer navigation to distinguish internal routes from external and mailto links.

Changes

Policy pages and navigation

Layer / File(s) Summary
Policy page content
src/pages/PrivacyPage.jsx, src/pages/TermsPage.jsx
Adds static policy pages with home navigation, headings, and informational sections.
Route and footer wiring
src/App.jsx, src/components/layout/Footer.jsx
Registers the policy routes and renders internal links with Link. External and mailto links use secure anchors that open in a new tab.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to 35300

This PR adds the Terms of Service and Privacy Policy pages and connects the footer links to them. No actionable merge-blocking risk remains beyond normal checks and review.

Possibly related issues

  • Issue 106 — Adds the /terms and /privacy routes needed for footer policy links.

Possibly related PRs

Suggested labels: Typescript Lang

Suggested reviewers: rahul-vyas-dev

Poem

A rabbit hops through routes so neat,
Terms and Privacy pages now meet.
Footer links know where to go,
Secure paths in tidy flow.
Hop, hop—policy is complete! 🐇

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the addition of the Terms of Service and Privacy Policy pages.
Linked Issues check ✅ Passed The PR adds both pages, registers their routes, and updates footer navigation to resolve issue #114.
Out of Scope Changes check ✅ Passed The changes support the linked issue and PR objectives; no unrelated code changes are evident.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Warning

⚠️ This pull request shows signs of AI-generated slop (description_diff_mismatch). It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

@github-actions github-actions Bot added first-time-contributor First time contributor size/L 201-500 lines changed and removed size/L 201-500 lines changed labels Aug 20, 2026
@gitcordapp

gitcordapp Bot commented Aug 20, 2026

Copy link
Copy Markdown

Link your account with Gitcord

Thanks for opening this PR, @prathamkhatwani!

To receive Discord notifications and contributor tracking for this organization:

  1. Join Discord: https://discord.gg/hjUhu33uAn
  2. In Discord, run /link prathamkhatwani
  3. Paste the verification code into your GitHub bio (or a public gist)
  4. Click Verify in Discord (or run /verify-link prathamkhatwani)

Once linked, Gitcord can notify you about reviews, merges, and more.

Posted by Gitcord

@Ri1tik

Ri1tik commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

We already have worked on this. Please see all the issues and PR before starting working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working first-time-contributor First time contributor frontend Frontend changes javascript JavaScript/TypeScript changes size/L 201-500 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Adding the "Terms of service" and "Privacy policy" page

2 participants