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.
Description
As recommended by Erik Wiener:
I see the use of "notification" to refer to proposition interactions or events in the docs and in the code, e.g. sendDisplayNotifications and includePendingDisplayNotifications. I personally think it would be better to avoid the term "notification", which is a Target concept, and instead try to align around "proposition", "proposition interaction", and "event", which are solution-agnostic terms. Those two new parameters above stick out to me because I don't think we've tended to use the term "notification" in public docs related to personalization (though I could be wrong). I feel it would be more consistent to use names like sendDisplayEvent and includeRenderedPropositions.
I think of the whole event as the notification, and the proposition interactions inside _experience.decisioning.propositions as the experiences that are the subject of the event (i.e. what personalization experiences were interacted with as part of that event).
Related Issue
Motivation and Context
Screenshots (if appropriate):
Types of changes
Checklist: