Skip to content

Apply grab-deps asset registration pattern#88

Merged
fumikito merged 1 commit intomasterfrom
fix/grab-deps-assets
Mar 12, 2026
Merged

Apply grab-deps asset registration pattern#88
fumikito merged 1 commit intomasterfrom
fix/grab-deps-assets

Conversation

@fumikito
Copy link
Contributor

Summary

  • Add @handle/@deps/@footer headers to JS source files for grab-deps
  • Implement wp-dependencies.json-based asset registration via cookie_tasting_register_assets()
  • Simplify wp_enqueue_script/style calls to handle-only (remove manual wp_register_script with hardcoded paths/deps/versions)
  • Remove load_plugin_textdomain call (translations handled by GlotPress)
  • Remove committed wp-dependencies.json (now a build artifact, added to .gitignore)
  • Commit package-lock.json (removed from .gitignore)
  • Update deploy.yml to use npm run package instead of bash bin/build.sh

🤖 Generated with Claude Code

Co-authored-by: Claude <noreply@anthropic.com>
@fumikito fumikito enabled auto-merge March 12, 2026 07:37
@fumikito fumikito merged commit 3ca9c6b into master Mar 12, 2026
8 checks passed
@fumikito fumikito deleted the fix/grab-deps-assets branch March 12, 2026 07: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.

1 participant