Thanks to visit codestin.com
Credit goes to github.com

Skip to content

docs: disclose Premium TTS data flow in production privacy policy (en+ja)#83

Merged
Saqoosha merged 1 commit into
mainfrom
privacy-tts-disclosure
May 28, 2026
Merged

docs: disclose Premium TTS data flow in production privacy policy (en+ja)#83
Saqoosha merged 1 commit into
mainfrom
privacy-tts-disclosure

Conversation

@Saqoosha

Copy link
Copy Markdown
Owner

What

Updates the production (main-served) privacy policy to disclose the HDZap Premium TTS data flow, and adds the missing Japanese privacy page.

Why

saqoosha.github.io/HDZap/privacy/ is served from main, which was last released (PR #72, 2026-05-19) before Premium TTS existed. So production still serves the original pre-Premium privacy page:

  • It does not disclose that, with Premium enabled, lap-announcement text is sent to AWS Polly / Azure for cloud TTS.
  • /privacy/ja/ returns 404 (the JA translation only ever existed on develop).

HDZap 1.1.0 (build 20, currently in App Review) introduces Premium and its paywall links to this page. App Review already rejected once on Guideline 3.1.2(c) for the privacy link; the link itself is now fixed (build 20 → 200), but the content on production must accurately describe the data flow.

Scope

Privacy pages only — cherry-picked from develop:

  • docs/manual/_pages/privacy/index.html — adds the Premium TTS disclosure (what is/isn't sent, 30-day audio cache, StoreKit JWS); Polly + Azure only (Cartesia removed, no longer used)
  • docs/manual/_pages/privacy/ja/index.html — new JA page (fixes the /privacy/ja/ 404)

Manual (en.md / ja.md) and firmware are intentionally not touched — they correctly reflect the released 1.0.1. This does not affect free users.

🤖 Generated with Claude Code

…+ja)

Bring the live (main) privacy policy in line with HDZap Premium (1.1.0):
disclose that lap-announcement text is sent to AWS Polly / Azure for cloud
TTS, what is and isn't sent, the 30-day audio cache, and the StoreKit JWS.
Production previously served the pre-Premium privacy page with no TTS
disclosure. Manual and app are unchanged (still reflect released 1.0.1).

Co-Authored-By: Claude Opus 4.7 <[email protected]>
@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: d7a0bf7f-ace1-46f8-afd9-e3c4146c79e9

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch privacy-tts-disclosure

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 and usage tips.

@Saqoosha Saqoosha merged commit e0d7310 into main May 28, 2026
3 checks passed
@Saqoosha Saqoosha deleted the privacy-tts-disclosure branch May 28, 2026 01:36
Saqoosha added a commit that referenced this pull request May 28, 2026
main's flasher.yml staged only the English privacy page, so the JA page
(docs/manual/_pages/privacy/ja/index.html, added in #83) was never copied
into _site — /privacy/ja/ returned 404 on production, and the "日本語"
language switcher on the English privacy page was a dead link. Add the
conditional ja copy step, matching develop's flasher.yml.

Co-authored-by: Claude Opus 4.7 <[email protected]>
Saqoosha added a commit that referenced this pull request May 29, 2026
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