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

Skip to content

Added filter for loading pollify js global object#18

Merged
sabbir1991 merged 11 commits into
developfrom
fix/improvement-logics
Jun 13, 2026
Merged

Added filter for loading pollify js global object#18
sabbir1991 merged 11 commits into
developfrom
fix/improvement-logics

Conversation

@sabbir1991

@sabbir1991 sabbir1991 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Closes

Closes #

Summary

Test in WP Playground

▶ Test in WP Playground

How to Test

WordPress Security Checklist

  • Nonces verified (wp_verify_nonce)
  • Input sanitized (sanitize_text_field, absint, etc.)
  • Output escaped (esc_html, esc_attr, esc_url, wp_kses_post)
  • Capability checks (current_user_can)
  • DB queries use $wpdb->prepare()

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactor / code cleanup

Risks / Limitations

Checklist

  • All CI checks pass (lint, PHPCS, PHPUnit)
  • Tested via WP Playground or locally
  • CHANGELOG.md / version updated if needed

… in CI

- Remove vendor/, build/, assets/build/ from git tracking (.gitignore already excluded them); they are now generated on the fly in CI and locally
- Rework build-assets.yml into 'Playground Build': builds JS assets, installs production composer deps, creates plugin zip via wp-scripts plugin-zip, uploads it as a workflow artifact
- Playground PR link now installs the built artifact through playground.wordpress.net/plugin-proxy.php instead of the raw branch zip, so PRs are testable without committing build output
The npm build script chains 'npm run makepot' which requires WP-CLI,
not available on the CI runner (exit 127: wp not found). The Playground
test zip does not need a regenerated .pot file, so run only the
wp-scripts build and admin JS build steps.
playground.wordpress.net/plugin-proxy.php rejects non-allowlisted orgs,
so the artifact-proxy approach cannot work for this repo. Instead the
workflow now pushes the built zip to a dedicated playground-builds
branch (kept at a single orphan commit) and links Playground to the
raw.githubusercontent.com URL, which serves the required CORS headers.
A commit-sha cache-buster in the URL ensures each PR push gets a link
to that exact build.
- release.yml: on GitHub Release publish, verify tag matches plugin
  header and readme stable tag, build JS assets + pot file, install
  production composer deps, deploy to wp.org SVN via
  10up/action-wordpress-plugin-deploy and attach the zip to the
  release; manual dispatch supports dry-run
- asset-update.yml: sync .wordpress-org assets and readme.txt to the
  SVN assets dir on push to master via
  10up/action-wordpress-plugin-asset-update
- .distignore: authoritative exclude list for the deployed package
- .wordpress-org/: banners, icons and screenshots tracked in repo as
  the source of truth for wp.org store assets
- Remove legacy .svnignore (replaced by .distignore)
- Ship uninstall.php in plugin zip (was missing from package.json files)
github-actions Bot added a commit that referenced this pull request Jun 12, 2026
sabbir1991 and others added 2 commits June 12, 2026 20:05
The original untracking commit lost its deletions during a rebase;
this restores the intended state. vendor/, build/ and assets/build/
are generated by CI and local builds, and are already in .gitignore.

Co-Authored-By: Claude Fable 5 <[email protected]>
github-actions Bot added a commit that referenced this pull request Jun 12, 2026
github-actions Bot added a commit that referenced this pull request Jun 13, 2026
github-actions Bot added a commit that referenced this pull request Jun 13, 2026
@sabbir1991 sabbir1991 merged commit c0873c6 into develop Jun 13, 2026
7 checks passed
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