Skip to content

updated listrak documentation #5014

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 9 commits into from
Jul 12, 2023

Conversation

jon-listrak
Copy link
Contributor

Proposed changes

We updated our documentation to better reflect Listrak's initial source offering.

Merge timing

ASAP once approved

Related issues (optional)

@jon-listrak jon-listrak requested a review from a team as a code owner July 12, 2023 17:17
@jon-listrak jon-listrak requested review from forstisabella and removed request for a team July 12, 2023 17:17
4. Click **Add Source** to save your settings.
5. Copy the Write key from the Segment UI and [log in to your Listrak account](https://admin.listrak.com) - navigate to Integrations > Integrations Management
6. Click New Integration
7. Click the Segment card
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the button in the UI Setup Integration or Set up integration?

Suggested change
7. Click the Segment card
7. Select the Segment card and click **Setup Integration**.

Copy link
Contributor

Choose a reason for hiding this comment

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

The steps here are correct. First you click the Segment card which takes you to a new page. Then you click the "Setup Integration" button on that page.

Copy link
Contributor

Choose a reason for hiding this comment

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

👍

5. Copy the Write key from the Segment UI and [log in to your Listrak account](https://admin.listrak.com) - navigate to Integrations > Integrations Management
6. Click New Integration
7. Click the Segment card
8. Click Setup Integration
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
8. Click Setup Integration

Copy link
Contributor

Choose a reason for hiding this comment

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

The steps here are correct. First you click the Segment card which takes you to a new page. Then you click the "Setup Integration" button on that page.

Copy link
Contributor

Choose a reason for hiding this comment

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

I removed this line of the docs because this information was added to the preceding step.

Copy link
Contributor

Choose a reason for hiding this comment

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

I saw that, but since I didn't apply that change, this change also should not be applied.

@forstisabella forstisabella self-requested a review July 12, 2023 19:01
@forstisabella forstisabella merged commit 357a306 into segmentio:develop Jul 12, 2023
@github-actions
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.

@forstisabella forstisabella added the fixed-content fix to content that has outdated or wrong information label Jul 12, 2023
@gary-hopkins-listrak gary-hopkins-listrak deleted the listrak/source-update branch July 13, 2023 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed-content fix to content that has outdated or wrong information
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants