Skip to content

docs: document the external services the bundled editor can reach - #126

Open
erseco wants to merge 2 commits into
mainfrom
feature/document-external-services
Open

erseco wants to merge 2 commits into
mainfrom
feature/document-external-services

Conversation

@erseco

@erseco erseco commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Document external services and remote resources used by the bundled editor, including their purpose, trigger, browser data and provider policies.

The disclosure covers authoring, preview and published content, including content created by other authors. It distinguishes local ELPX storage from browser requests and avoids claiming that no user/site data leaves the browser or that every provider is account-free. GeoGebra sends a material identifier. H5P policies depend on the actual host. JW Player loads when the interactive-video editor opens. Legacy lightbox share widgets are disabled by the normal initializer and send the page URL if enabled.

H5P privacy now links to the policy linked by h5p.org itself. EducaMadrid links to Mediateca's own legal/privacy/cookie notice.

This documentation does not resolve the prohibition on non-service CDN code. Reconcile the entries against the actual editor build before submission: the upstream tree already removes the obsolete mindmap HTML and changes MathJax packaging, while the currently installed dist/static still contains the older resources. No claim is made that pending upstream vendoring/upgrades are complete.

Validation: reviewed against the local editor and installed bundle; checked the corrected policy destinations; git diff --check passed. No runtime changes in this PR.

WordPress.org requires every third-party service a plugin can contact to be
named in the readme, with what is sent, when, and links to its terms and privacy
policy. The plugin itself contacts nothing; the bundled editor can, but only for
content the author deliberately builds with it -- a YouTube video, a GeoGebra
activity, an H5P embed, a molecule loaded from the RCSB PDB. Each entry says
which iDevice triggers it and what leaves the browser.
@github-actions

Copy link
Copy Markdown
Contributor

Test in WordPress Playground

Test the plugin with the code from this branch:

Preview in WordPress Playground

ℹ️ The eXeLearning editor is fetched from the shared release and unpacked into the plugin when the playground boots, so the first load may take a few extra seconds. ELP upload, shortcode, Gutenberg block and preview work normally.

@codecov-commenter

codecov-commenter commented Sep 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.87%. Comparing base (97fe8af) to head (482afa3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #126   +/-   ##
=========================================
  Coverage     96.87%   96.87%           
  Complexity      864      864           
=========================================
  Files            39       39           
  Lines          4323     4323           
=========================================
  Hits           4188     4188           
  Misses          135      135           
Flag Coverage Δ
javascript 95.70% <ø> (ø)
php 97.25% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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