-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update A4T section to provide further clarification on server and client side logging #409
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just a small comment.
|
||
For this integration to work, make sure Analytics is enabled in your datastream configuration for the desired environment and Report Suite information is provided. | ||
Server-side logging: If Analytics is enabled and configured in your datastream for the desired environment, then it is considered server-side logging. In this case, the Experience Edge handles forwarding any Target A4T payloads to Adobe Analytics, upon tracking method calls, and no analytics tokens are returned to the client.<br/>Client-side logging: If Analytics is disabled in your datastream for the desired environment, then it is considered client-side logging. In this case, analytics tokens are returned to the client and it is the responsibility of the customer to extract and send the data to Adobe Analytics, if desired. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For Analytics tokens - are these tokens from Adobe Analytics? If it is, it should be "... and no Analytics tokens are returned..." and "... this case, Analytics tokens are returned..."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: can we bold "Server-side logging" and "Client-side logging"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@calebk1 The tokens returned are meant for AA, I'll fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the updates @swarna04 🎉
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: