Terms of Service, Privacy Policy, and Copyright update#466
Terms of Service, Privacy Policy, and Copyright update#466AnthonyRonning merged 1 commit intomasterfrom
Conversation
Deploying maple with
|
| Latest commit: |
b239a9a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://53a1bba0.maple-ca8.pages.dev |
| Branch Preview URL: | https://tosupdates.maple-ca8.pages.dev |
📝 WalkthroughWalkthroughUpdated company legal name in public LLM metadata, added internal Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
frontend/src/components/SimplifiedFooter.tsx (1)
32-43: Consider using TanStack Router'sLinkcomponent for internal routes.The
/termsand/privacylinks use standard anchor tags, whileFooter.tsxuses TanStack Router'sLinkcomponent for the same routes. UsingLinkprovides client-side navigation benefits (no full page reload, prefetching support).Note: The existing
/downloadslink also uses an anchor tag, so this may be an intentional pattern for this component. If so, this is fine to keep as-is since the routes work correctly either way.♻️ Optional: Use Link component for consistency
+import { Link } from "@tanstack/react-router"; import { isTauri } from "@/utils/platform"; // ... <div className="flex justify-center gap-6 mt-2"> - <a + <Link - href="/downloads" + to="/downloads" className="text-[hsl(var(--marketing-text-muted))] hover:text-foreground text-sm transition-colors" > Downloads - </a> - <a + </Link> + <Link - href="/terms" + to="/terms" className="text-[hsl(var(--marketing-text-muted))] hover:text-foreground text-sm transition-colors" > Terms of Service - </a> - <a + </Link> + <Link - href="/privacy" + to="/privacy" className="text-[hsl(var(--marketing-text-muted))] hover:text-foreground text-sm transition-colors" > Privacy Policy - </a> + </Link> </div>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@frontend/src/components/SimplifiedFooter.tsx` around lines 32 - 43, The SimplifiedFooter component currently uses plain anchor tags for internal routes (/terms and /privacy); replace those anchors with TanStack Router's Link component (e.g., <Link to="/terms"> and <Link to="/privacy">) to enable client-side navigation and prefetching, and add the corresponding import for Link at the top of SimplifiedFooter.tsx; mirror the usage pattern from Footer.tsx so styling classes remain the same and only the tag and prop name change.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@frontend/src/components/SimplifiedFooter.tsx`:
- Around line 32-43: The SimplifiedFooter component currently uses plain anchor
tags for internal routes (/terms and /privacy); replace those anchors with
TanStack Router's Link component (e.g., <Link to="/terms"> and <Link
to="/privacy">) to enable client-side navigation and prefetching, and add the
corresponding import for Link at the top of SimplifiedFooter.tsx; mirror the
usage pattern from Footer.tsx so styling classes remain the same and only the
tag and prop name change.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: de94e5ce-0a3f-4a11-b055-ae2ff619c201
⛔ Files ignored due to path filters (1)
frontend/public/mark.jpgis excluded by!**/*.jpg
📒 Files selected for processing (8)
frontend/public/llms-full.txtfrontend/public/llms.txtfrontend/src/components/Footer.tsxfrontend/src/components/SimplifiedFooter.tsxfrontend/src/routeTree.gen.tsfrontend/src/routes/about.tsxfrontend/src/routes/privacy.tsxfrontend/src/routes/terms.tsx
|
there are likely links throughout to our opensecret link that need to be changed over as well. |
|
amend changes, not two seperate commits. |
- Add /privacy and /terms routes with content adapted to the site's design system - Link to the new pages from Footer and SimplifiedFooter (replacing external opensecret.cloud links) and from AccountMenu - Rebrand legal entity from Mutiny Wallet Inc to Maple Privacy Labs Inc in footers, about page, and llms.txt files - Remove the 'Powered by OpenSecret' attribution from footers - Update mark.jpg
There was a problem hiding this comment.
🧹 Nitpick comments (1)
frontend/src/routes/privacy.tsx (1)
31-43: Consider usingcn()utility or template literal for the long className.The current string concatenation works but could be slightly more readable. This is a minor style preference.
♻️ Optional: Using template literal
<article - className={ - "flex flex-col gap-4 dark:border-white/10 " + - "border-[hsl(var(--marketing-card-border))] dark:bg-black/75 " + - "bg-[hsl(var(--marketing-card))]/80 dark:text-white p-8 border rounded-lg " + - "[&_h2]:text-2xl [&_h2]:font-medium [&_h2]:mt-6 [&_h2]:mb-2 " + - "[&_p]:leading-relaxed " + - "[&_ul]:list-disc [&_ul]:pl-6 [&_ul]:space-y-2 [&_li]:leading-relaxed " + - "[&_a]:text-[hsl(var(--purple))] dark:[&_a]:text-[hsl(var(--blue))] " + - "[&_a]:underline hover:[&_a]:no-underline " + - "[&_strong]:font-semibold" - } + className={` + flex flex-col gap-4 dark:border-white/10 + border-[hsl(var(--marketing-card-border))] dark:bg-black/75 + bg-[hsl(var(--marketing-card))]/80 dark:text-white p-8 border rounded-lg + [&_h2]:text-2xl [&_h2]:font-medium [&_h2]:mt-6 [&_h2]:mb-2 + [&_p]:leading-relaxed + [&_ul]:list-disc [&_ul]:pl-6 [&_ul]:space-y-2 [&_li]:leading-relaxed + [&_a]:text-[hsl(var(--purple))] dark:[&_a]:text-[hsl(var(--blue))] + [&_a]:underline hover:[&_a]:no-underline + [&_strong]:font-semibold + `} >🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@frontend/src/routes/privacy.tsx` around lines 31 - 43, The long concatenated className string on the article element should be made more readable by using the cn() utility or a template literal: import or reference your existing cn/classnames helper and replace the concatenated string assigned to the className prop on the article element (the large "flex flex-col ..." string) with a single cn(...) call (or a backtick template literal), grouping related classes for clarity; ensure you keep all original classes and preserve conditional/dark mode classes exactly as-is when migrating.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@frontend/src/routes/privacy.tsx`:
- Around line 31-43: The long concatenated className string on the article
element should be made more readable by using the cn() utility or a template
literal: import or reference your existing cn/classnames helper and replace the
concatenated string assigned to the className prop on the article element (the
large "flex flex-col ..." string) with a single cn(...) call (or a backtick
template literal), grouping related classes for clarity; ensure you keep all
original classes and preserve conditional/dark mode classes exactly as-is when
migrating.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 3da3ac40-ac13-4305-b59b-1fdb91ead8b1
⛔ Files ignored due to path filters (1)
frontend/public/mark.jpgis excluded by!**/*.jpg
📒 Files selected for processing (8)
frontend/public/llms-full.txtfrontend/public/llms.txtfrontend/src/components/AccountMenu.tsxfrontend/src/components/Footer.tsxfrontend/src/components/SimplifiedFooter.tsxfrontend/src/routeTree.gen.tsfrontend/src/routes/privacy.tsxfrontend/src/routes/terms.tsx
✅ Files skipped from review due to trivial changes (5)
- frontend/public/llms-full.txt
- frontend/public/llms.txt
- frontend/src/components/AccountMenu.tsx
- frontend/src/components/SimplifiedFooter.tsx
- frontend/src/routeTree.gen.ts
🚧 Files skipped from review as they are similar to previous changes (1)
- frontend/src/routes/terms.tsx
Add Privacy and Terms pages
Summary
styled to match the rest of the marketing site (TopNav, FullPageMain, MarketingHeader,
marketing card layout).
Maple Privacy Labs Inc. (formerly Mutiny Wallet Inc), and the company is based in San
Francisco and Austin.
SimplifiedFooter with the new local routes, and adds Privacy/Terms links to the main
Footer's Resources column.
Notes
typography rather than @tailwindcss/typography, which isn't installed in this project.
processing purposes, and user data rights, to make the content easier to scan.
Test plan
all render correctly.
Summary by CodeRabbit
New Features
Updates