Skip to content

fix: improvement the privacy message in all languages#532

Merged
joanise merged 2 commits intomainfrom
dev.ej/fix-privacy-wording
Mar 6, 2026
Merged

fix: improvement the privacy message in all languages#532
joanise merged 2 commits intomainfrom
dev.ej/fix-privacy-wording

Conversation

@joanise
Copy link
Member

@joanise joanise commented Mar 6, 2026

PR Goal?

  • the fact that we have to pay for Playsible is not relevant, that's our problem
  • "which devices" feels creepy to me, like we're tracking indivual devices. I already worded it as types of devices in French and Spanish when I translated, and here I am backporting that fix to English too.

Feedback sought?

Do you like the new text?

To see the differences more clearly, use the SemanticDiff link, there are really just small changes which the github diff fails to highlight well.

Priority?

low

How to test?

look at the PR preview

Confidence?

high

Version change?

no

@joanise joanise requested a review from deltork March 6, 2026 19:47
@semanticdiff-com
Copy link

semanticdiff-com bot commented Mar 6, 2026

Review changes with  SemanticDiff

Changed Files
File Status
  packages/studio-web/src/app/privacy-dialog.html  95% smaller
  packages/studio-web/src/i18n/messages.json  91% smaller
  packages/studio-web/src/i18n/messages.es.json  1% smaller
  packages/studio-web/src/i18n/messages.fr.json  0% smaller
  update-translations.sh Unsupported file format

@joanise
Copy link
Member Author

joanise commented Mar 6, 2026

Oh, how disappointing! SemantifDiff did not pick up on the fact that the massive line in French and Spanish has just a few small changes.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

PR Preview Action v1.6.3

🚀 View preview at
https://ReadAlongs.github.io/Studio-Web/pr-preview/pr-532/

Built to branch gh-pages at 2026-03-06 20:43 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@dlothian
Copy link

dlothian commented Mar 6, 2026

In the translation you gave me for the WW blurb you included "(en anglais)" but not in this one. Should it be here too?

@joanise
Copy link
Member Author

joanise commented Mar 6, 2026

In the translation you gave me for the WW blurb you included "(en anglais)" but not in this one. Should it be here too?

You know, it's not really necessary, and it takes more space for little value.

- the fact that we have to pay for Playsible is not relevant, that's our
  problem
- "which devices" feels creepy to me, like we're tracking indivual
  devices. I already worded it as types of devices in French and Spanish
  when I translated, and here I am backporting that fix to English too.
@joanise joanise force-pushed the dev.ej/fix-privacy-wording branch from 1ab84e0 to 150f451 Compare March 6, 2026 20:38
@joanise joanise merged commit 150f451 into main Mar 6, 2026
3 checks passed
@joanise joanise deleted the dev.ej/fix-privacy-wording branch March 6, 2026 20:39
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