Skip to content

Conversation

ksaha873
Copy link
Contributor

Proposed changes

Adding warning for null segment_id in Track event tables in cases where ID resolution has failed. Doing this because of a request for change in documentation following this issue: Null segment_ids in tables.

Merge timing

ASAP once approved.

Related issues (optional)

JIRA link: Null segment_ids in tables.

### What: 

Adding warning for null ```segment_id``` in Track event tables in cases where ID resolution has failed.

### Why:

Request for change in documentation following this issue: [Null segment_ids in tables](https://twilio-engineering.atlassian.net/browse/PS-403).
@ksaha873 ksaha873 requested review from pwseg and a team as code owners May 28, 2025 15:21
@pwseg pwseg changed the title Update tables.md Add note about segment_id behavior in Track event tables where ID res fails May 29, 2025
@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label May 29, 2025
@pwseg pwseg merged commit ed3ac0b into develop May 29, 2025
4 checks passed
@pwseg pwseg deleted the ksaha873-patch-1 branch May 29, 2025 21:21
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