Fix WordPress.org compliance issues#4
Open
bwstefano wants to merge 5 commits intocodex/i18n-string-reviewfrom
Open
Fix WordPress.org compliance issues#4bwstefano wants to merge 5 commits intocodex/i18n-string-reviewfrom
bwstefano wants to merge 5 commits intocodex/i18n-string-reviewfrom
Conversation
8302279 to
705e1c1
Compare
Document third-party services, tighten escaping and settings sanitization, and remove avoidable remote asset dependencies for the WordPress.org review path. This also restores editor behavior that regressed during the compliance pass, including geolocation search UX, storymap search alignment, legend rendering, embed logo sizing, and translation consistency for Discovery embed actions. Co-authored-by: Codex <codex@openai.com>
Run the release workflow only for stable tags, stage the built src tree as jeowp for Plugin Check, and block WordPress.org deployment until that validation passes. Document the new release gate in the README so the stable tagging expectations stay aligned with the CI pipeline. Co-authored-by: Codex <codex@openai.com>
Bring the WordPress.org remediation work to a shippable state by documenting third-party services, hardening escaping and settings handling, and restoring the editor and Discovery flows that regressed during the compliance pass. This updates map and storymap editing, Discovery layer loading, legend rendering, embed output, geocoding search UX, translations, REST helpers, and package metadata so the plugin is ready for the stacked PR review. Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
- harden Related Posts loading, clustering, and runtime compatibility across Mapbox and MapLibre - make Discovery pagination, hover behavior, and story deduplication more predictable - fix storymap navigation edge cases and improve aligned map block behavior in editor and frontend - keep map overlays responsive on smaller screens and add regression helpers/tests for REST query serialization Co-authored-by: Codex <codex@openai.com>
ae5000c to
fb20d0a
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Context
This PR is intentionally stacked on top of
codex/i18n-string-reviewand should be reviewed after that PR.The goal here is to address the WordPress.org review concerns around:
What Changed
Third-party services and privacy
Remote dependencies and compliance hardening
Escaping, sanitization, and rendering
Geocoding and editor behavior
Legends and translations
Embedin both Portuguese and Spanish, while keeping translated wording in other contextspt_BRandes_COValidation
npm run check:envnpm run buildnpm run build:reportnpm run test:unitcomposer validate --no-check-publishvendor/bin/phpcs --standard=phpcs.xml.distphp scripts/check-php-compat.phpNotes
wp-clistill emits deprecation warnings duringi18n:json, but the build completes successfully and the warnings come from the toolchain, not plugin runtimemapBlocksbundle budget was updated to reflect the current stable CKEditor footprint used by the storymap editor