Skip to content
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

Add customer header and keepalive info to Analytics.js documentation #7369

Merged
merged 4 commits into from
Jan 23, 2025

Conversation

silesky
Copy link
Contributor

@silesky silesky commented Jan 2, 2025

  • Added info on configuring custom headers using deliveryStrategy.config.headers in Analytics.js.
  • Clarified when Segment uses keepalive by default (page unload events and batching enabled).

@silesky silesky requested a review from a team as a code owner January 2, 2025 18:39
@silesky silesky requested a review from pwseg January 2, 2025 18:39
@silesky silesky changed the title Ipdate ajs settings Update ajs settings Jan 2, 2025
@pwseg pwseg changed the title Update ajs settings Add customer header and keepalive info to Analytics.js documentation Jan 23, 2025
@pwseg
Copy link
Contributor

pwseg commented Jan 23, 2025

@silesky I made some changes to this new info.

Specifically, I created a new section to talk about delivery strategy in general. I also add some more specifics to when Segment uses keepalive, and also threw semi-colons in the end of the code snippets; let me know if you want any of this changed.

@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Jan 23, 2025
@pwseg pwseg merged commit 7ae64ea into develop Jan 23, 2025
4 checks passed
@pwseg pwseg deleted the add-ajs-settings-info branch January 23, 2025 20:26
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content-update updates to content that are not new features, includes grammar fixes, added notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants