Skip to content

Update track.md - Track Spec added context.traits and actions destina… #5982

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
Apr 12, 2024

Conversation

lizkane222
Copy link
Contributor

Proposed changes

Customers do not always know that it's possible to include traits in Track events or that even the same event can trigger multiple Actions within a destination actions.

DOCS TEAM :
Could you please assist in the formatting of the table I added, which I just copied from this existing doc?
Spec : Common Fields
Edit Mode

Merge timing

  • ASAP once approved?

Related issues (optional)

Zendesk ticket : https://segment.zendesk.com/agent/tickets/528339

@lizkane222 lizkane222 marked this pull request as ready for review February 29, 2024 22:45
@lizkane222 lizkane222 requested a review from a team as a code owner February 29, 2024 22:45
@lizkane222 lizkane222 requested review from rchinn1 and removed request for a team February 29, 2024 22:45

<table>
{% include content/spec-table-header.md %}
{% include content/spec-field-context.md %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lizkane222 - the formatting looks good. Is context the only field you wanted to display in the table?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lizkane222 WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2024-04-09 at 11 16 40

@cmastr cmastr requested review from rchinn1 and cmastr and removed request for rchinn1 March 20, 2024 17:51
@cmastr cmastr added the KCS label Mar 25, 2024
Copy link

netlify bot commented Apr 12, 2024

Deploy Preview for segment-docs ready!

Name Link
🔨 Latest commit 665eef8
🔍 Latest deploy log https://app.netlify.com/sites/segment-docs/deploys/661959e7d7cad10007dfd19e
😎 Deploy Preview https://deploy-preview-5982--segment-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cmastr cmastr merged commit 674e65b into develop Apr 12, 2024
@cmastr cmastr deleted the lizkane222-patch-16 branch April 12, 2024 18:48
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