Skip to content

feat: FAQ page with FAQPage schema - #33

Merged
KMKoushik merged 1 commit into
mainfrom
hq/growth-faq
Oct 14, 2026
Merged

KMKoushik merged 1 commit into
mainfrom
hq/growth-faq

Conversation

@KMKoushik

Copy link
Copy Markdown
Contributor

What changed

  • New public page /faq answering five questions: is it free, limits, offline, currencies, export. It reuses the existing Faq block and faqJsonLd, so the page has FAQPage structured data that matches the visible text.
  • /faq is added to sitemap.xml, so the "FAQ page in sitemap" check passes once this is deployed.
  • An FAQ link is added to the footer under Resources.

Where the answers come from

  • Free: same wording as the landing page. There's no billing code in the repo. I left out ads because they're configured per environment and are off in production right now.
  • Limits: MAX_AMOUNT_CENTS (21,474,836.47), FILE_SIZE_LIMIT (5 MB) and quick split's MAX_PEOPLE (20). I found no cap on groups or friends. The Splitwise "4 per day" figure is the same sourced claim the landing page and blog already make.
  • Offline: next-pwa makes the app installable, but changes go through tRPC and there's no offline queue, so the page says no. Users can pick a past date when they add an expense.
  • Currencies: "more than 100" (119 in currency.ts). Balances are kept per currency, groups have a default currency, there's no exchange-rate conversion, and a quick split uses one currency.
  • Export: the CSV export on a friend's page (getExpensesWithFriend includes expenses in shared groups) and the JSON "Download splitpro data" on the Account page (friends, groups, balances). The page says there's no group CSV export yet.

How I checked it

  • npx prettier --write, npx tsc --noEmit -p . (clean) and SKIP_ENV_VALIDATION=1 pnpm lint (no new warnings)
  • On a local next dev: /faq returns 200 with the FAQPage JSON-LD, and /sitemap.xml lists https://splitpro.app/faq

@coderabbitai

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 58 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1d49c0f6-021d-420f-850b-cdc38774ffd5
📥 Commits

Reviewing files that changed from the base of the PR and between 5946d5a and 59ef6ae.

📒 Files selected for processing: 3
  • src/components/Site/SiteLayout.tsx
  • src/pages/faq.tsx
  • src/pages/sitemap.xml.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@KMKoushik
KMKoushik merged commit 2fa45cc into main Oct 14, 2026
2 checks passed
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.

1 participant