Reader Comments: Track when notification setting is toggled #17369
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.
Refs #17363 (comment)
Depends on #17340
This adds a new event,
reader_toggle_comment_notifications
, which tracks whenever the comments notification setting is toggled. In addition, these properties are added:blog_id
: The site ID of the blog that the user enabled the notifications on.notifications_enabled
:true
if the notifications is enabled, orfalse
otherwise.To test:
Make sure the "Follow Conversation via Notifications" flag is enabled. You should see
🔵 Tracked: reader_toggle_comment_notifications
when:Regression Notes
Potential unintended areas of impact
n/a. Feature is hidden behind a flag.
What I did to test those areas of impact (or what existing automated tests I relied on)
n/a. Feature is hidden behind a flag.
What automated tests I added (or what prevented me from doing so)
n/a. Feature is hidden behind a flag.
PR submission checklist:
RELEASE-NOTES.txt
if necessary.