Skip to content

fix: privacy and terms pages links#815

Merged
graphite-app[bot] merged 1 commit into
mainfrom
03-30-fix_privacy_and_terms_pages_links
Mar 30, 2026
Merged

fix: privacy and terms pages links#815
graphite-app[bot] merged 1 commit into
mainfrom
03-30-fix_privacy_and_terms_pages_links

Conversation

@MaheshtheDev
Copy link
Copy Markdown
Member

No description provided.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
🔵 In progress
View logs
supermemory-mcp 52147c2 Mar 30 2026, 05:07 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Mar 30, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 13d69aa Mar 30 2026, 05:15 PM

Copy link
Copy Markdown
Member Author


How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@MaheshtheDev MaheshtheDev marked this pull request as ready for review March 30, 2026 17:08
@claude
Copy link
Copy Markdown

claude Bot commented Mar 30, 2026

Claude finished @MaheshtheDev's task in 48s —— View job


Review Complete

  • Fetch and analyze PR diff
  • Check for bugs, security issues, and logic errors
  • Submit review

Overview: Updates the Terms and Privacy Policy URLs from /terms-of-service and /privacy-policy to /terms/ and /privacy/ in both the web app login page and the shared UI login component.

Issues found: None — this looks good to ship.

Score: 10/10

Simple URL path updates across two files. No logic changes, no security implications, no bugs. The changes are consistent between both files.

@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Mar 30, 2026

Merge activity

@graphite-app graphite-app Bot force-pushed the 03-30-fix_privacy_and_terms_pages_links branch from 52147c2 to 13d69aa Compare March 30, 2026 17:10
@graphite-app graphite-app Bot merged commit 13d69aa into main Mar 30, 2026
6 of 8 checks passed
@entelligence-ai-pr-reviews
Copy link
Copy Markdown


Confidence Score: 5/5 - Safe to Merge

Safe to merge — this PR makes a straightforward URL path update in two login page files, changing /terms-of-service to /terms/ and /privacy-policy to /privacy/ in both apps/web/app/(auth)/login/new/page.tsx and packages/ui/pages/login.tsx. The change is purely cosmetic in terms of functionality and carries no logic, security, or runtime risk beyond the correctness of the new URLs themselves. Both files are updated consistently, ensuring no divergence between the web app and shared UI package footer links.

Key Findings:

  • URL updates in apps/web/app/(auth)/login/new/page.tsx and packages/ui/pages/login.tsx are applied symmetrically, preventing any inconsistency between the two login page implementations.
  • The change is limited to static string literals in anchor href attributes — no logic, state, or runtime behavior is affected, making regression risk negligible.
  • No review comments or heuristic issues were flagged; all changed files were covered in the review with zero identified concerns.
Files requiring special attention
  • apps/web/app/(auth)/login/new/page.tsx
  • packages/ui/pages/login.tsx

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants