Skip to content

Update index.md #4716

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 2 commits into from
May 30, 2023
Merged

Update index.md #4716

merged 2 commits into from
May 30, 2023

Conversation

dineshraj-arunmozhi
Copy link
Contributor

Updating the document to have more clear wording

Proposed changes

Merge timing

Related issues (optional)

Updating the document to have more clear wording
@dineshraj-arunmozhi dineshraj-arunmozhi marked this pull request as ready for review May 11, 2023 16:31
@dineshraj-arunmozhi dineshraj-arunmozhi requested a review from a team as a code owner May 11, 2023 16:31
@dineshraj-arunmozhi dineshraj-arunmozhi requested review from stayseesong and removed request for a team May 11, 2023 16:31
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
@stayseesong stayseesong added the content-update updates to content that are not new features, includes grammar fixes, added notes label May 18, 2023
@@ -89,7 +89,7 @@ To add your first destination:
4. Select the Reverse ETL source you want to connect the destination to.
5. Enter the **Destination name** and click **Create Destination**.
6. Enter the required information on the **Settings** tab of the destination.
7. Enable the destination.
7. Navigate to the destination settings tab and enable the destination. If the destination is disabled, then Segment won't be able to start sync.
Copy link
Contributor

Choose a reason for hiding this comment

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

@dineshraj-arunmozhi curious why you want to add in that you have to navigate to the settings tab? From step 6, you should still be on the settings tab

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @stayseesong, We have customers writing to us stating their sync has not started. While investigating, we noticed every time it has to do with the enabling the destination. Additionally, a couple of customers had felt that is not clear enough for them.

@stayseesong stayseesong merged commit acf696f into develop May 30, 2023
@stayseesong stayseesong deleted the dineshraj-arunmozhi-patch-1 branch May 30, 2023 16:58
@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.

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