Skip to content

Deprecate ShopifyApp.add_csp_directives#2053

Open
lizkenyon wants to merge 1 commit intomainfrom
deprecate-add-csp-directives
Open

Deprecate ShopifyApp.add_csp_directives#2053
lizkenyon wants to merge 1 commit intomainfrom
deprecate-add-csp-directives

Conversation

@lizkenyon
Copy link
Contributor

Summary

  • Deprecates ShopifyApp.add_csp_directives(policy), targeting removal in v24.0.0
  • The method continues to function but now emits a deprecation warning via ShopifyApp::Logger.deprecated
  • Updates documentation and changelog to reflect the deprecation

Test plan

  • New test verifies deprecation warning is emitted with correct message and version
  • All 5 existing CSP helper tests continue to pass (method still works)
  • Full test suite passes (527 tests, 0 failures)

🤖 Generated with Claude Code

The method continues to function but now emits a deprecation warning
via ShopifyApp::Logger.deprecated. Updates docs, changelog, and adds
a test for the warning.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lizkenyon lizkenyon requested a review from a team as a code owner February 12, 2026 21:14
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