[UEPR-427] Wrong ToU and Privacy Policy links

[kbangelov]

Resolves:

https://scratchfoundation.atlassian.net/browse/UEPR-427

Changes:

Replaces all /terms_of_use with new link
all /privacy_policy with new link

Question:

Not clear to me whether those pages should be deleted since there should be no links leading to them left over. To be clarified.

[kbangelov]

Not sure whether this should be deleted as well.

[KManolov3]

Let's not change the routes

[kbangelov]

Also not sure if a link this long should appear in a text. Perhaps it should just say "Terms of Use"

[KManolov3]

Hmm, that's a good question. I think it's better if the text stays as it was for now, but still refers to the freshdesk link. In any case, this page shouldn't be accessible through the regular flow (and will only be accessible via url).

@cwillisf what do you think?

[rosics-code]

#9915

What about this? @kbangelov

[kbangelov]

#9915

What about this? @kbangelov

I don't fully comprehend the conflict of "Check for other places in the code, where Terms of Use or Privacy Policy leads to the internal page instead of Freshdesk and update them" and that PR which updates them, considering we want to replace all the links for the internal ToU page.

[Copilot]

Pull request overview

This PR updates Terms of Use and Privacy Policy links throughout the codebase to point to new Freshdesk URLs instead of internal routes. The changes also remove the corresponding route definitions since these pages are now hosted externally.

Key Changes:

• Replaced all /terms_of_use references with the new Freshdesk Terms of Use URL
• Replaced all /privacy_policy references with the new Freshdesk Privacy Policy URL
• Removed internal route definitions for these pages from routes.json

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/views/terms/terms.jsx	Updated Privacy Policy and Terms of Use links to external Freshdesk URLs
src/views/teachers/faq/faq.jsx	Updated Privacy Policy links in teacher FAQ
src/views/privacypolicy/privacypolicy.jsx	Updated Terms of Use link reference
src/views/privacypolicy-apps/privacypolicy-apps.jsx	Updated Privacy Policy and Terms of Use links
src/views/parents/parents.jsx	Updated Privacy Policy links in parents section
src/views/cookies/cookies.jsx	Updated all Privacy Policy link references in cookies table and text
src/routes.json	Removed route definitions for internal privacy policy and terms of use pages
src/redux/session.js	Removed /privacy_policy and /terms_of_use from ban exception list
src/components/privacy-banner/privacy-banner.jsx	Updated Privacy Policy link in banner component
src/components/footer/www/footer.jsx	Updated footer Terms of Use and Privacy Policy links

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[KManolov3]

This will address #9921 in making the links consistent (with the caveat, that freshdesk should be considered SoT, until the Terms of Use updates roll-out).

[KManolov3]

Let's use the external links object (here and in other places) instead of hardcoding them. You can reference how it was done for a subset of the places in - a904c31

[kbangelov]

will do

[KManolov3]

Hmm, that's a good question. I think it's better if the text stays as it was for now, but still refers to the freshdesk link. In any case, this page shouldn't be accessible through the regular flow (and will only be accessible via url).

@cwillisf what do you think?

[rosics-code]

Why not remove terms entirely? Terms of Use still mentions ScratchX which is now not working & it also has errors.

[cwillisf]

<td><a href={externalLinks.scratchHelpDesk.privacyPolicy}>looks</a></td>
<td><a href={externalLinks.scratchHelpDesk.privacyPolicy}>good</a></td>
<td><a href={externalLinks.scratchHelpDesk.privacyPolicy}>to</a></td>
<td><a href={externalLinks.scratchHelpDesk.privacyPolicy}>me</a></td>