Skip to content

Update custom-domain.md #6923

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

Merged
merged 3 commits into from
Aug 13, 2024
Merged

Update custom-domain.md #6923

merged 3 commits into from
Aug 13, 2024

Conversation

prigiattiperrut
Copy link
Contributor

Proposed changes

This update adds two new FAQ sections to the documentation page:

Benefit of Migrating Server-Side Sources to Custom Domain: Explains why server-side tracking is generally more reliable than client-side tracking and recommends keeping business-critical events server-side for more accurate and reliable tracking.

Configuring Non-JavaScript Sources to Use Custom Domain: Provides guidance on how to configure non-JavaScript sources to use Custom Domain for consistency, including specific instructions for updating implementations to use the subdomain as an endpoint when using the Tracking API.

Merge timing

ASAP once approved?

Related issues (optional)

Slack: https://twilio.slack.com/archives/CTTKLTS76/p1723156941435869

@prigiattiperrut prigiattiperrut requested a review from a team as a code owner August 9, 2024 11:30
@prigiattiperrut prigiattiperrut requested review from forstisabella, alanjcharles and atitsh and removed request for alanjcharles August 9, 2024 11:30
atitsh
atitsh previously approved these changes Aug 13, 2024
@forstisabella forstisabella merged commit f3324e1 into develop Aug 13, 2024
2 of 4 checks passed
@forstisabella forstisabella deleted the prigiattiperrut-patch-7 branch August 13, 2024 16:32
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants