breaking: remove deprecated removeCCPAState method#1237
breaking: remove deprecated removeCCPAState method#1237rmi22186 wants to merge 3 commits intoworkstation/3.0-Releasefrom
Conversation
Remove the deprecated `removeCCPAState` method and associated code/tests. Users should use `removeCCPAConsentState` instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
IConsentState was an empty extension of ConsentState that added no properties. Replace usages with ConsentState directly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The TODO indicated this cast should be removed once removeCCPAState was removed. Since removeCCPAState is being removed in this PR, the cast is no longer needed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
PR SummaryMedium Risk Overview Updates Written by Cursor Bugbot for commit 8f70c58. This will update automatically on new commits. Configure here. |
jaissica12
left a comment
There was a problem hiding this comment.
@rmi22186 The target branch is set to master. Shouldn't this target the workstation/3.0-Release branch instead, since it's a breaking change?
alexs-mparticle
left a comment
There was a problem hiding this comment.
LGTM but please make sure you point this to the v3 branch instead of master.
|
@alexs-mparticle @jaissica12 - good catch! Updated. |



Summary
removeCCPAStatemethod from the consent module. Users should useremoveCCPAConsentStateinstead.removeCCPAStateproperty from theIConsentStateinterface and associated TODO comments.removeCCPAState.Breaking Change
removeCCPAStatewas deprecated in favor ofremoveCCPAConsentState. This PR completes that deprecation by removing the method entirely. Any consumers still callingremoveCCPAStatemust migrate toremoveCCPAConsentState.Test plan
removeCCPAStatein source (only CHANGELOG history)🤖 Generated with Claude Code