Skip to content

AB Smartly Destination Docs #1777

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 27 commits into from
Aug 25, 2021
Merged

AB Smartly Destination Docs #1777

merged 27 commits into from
Aug 25, 2021

Conversation

gezgblack
Copy link
Contributor

Proposed changes

These are the destination docs for the new destination AB Smartly. The destination is currently in a "Private Beta" status but we'll be switching them to "Public Beta" as soon as the docs are approved.

Merge timing

Not in a massive hurry, but would be good to get reviewed and squared away.

@gezgblack gezgblack requested a review from markzegarelli August 3, 2021 09:11
@netlify
Copy link

netlify bot commented Aug 3, 2021

✔️ Deploy Preview for segment-docs ready!

🔨 Explore the source changes: febfc79

🔍 Inspect the deploy log: https://app.netlify.com/sites/segment-docs/deploys/61140643716dd30007b25b34

😎 Browse the preview: https://deploy-preview-1777--segment-docs.netlify.app

1. From the Destinations catalog page in the Segment App, click **Add Destination**.
2. Search for "A/B Smartly" in the Destinations Catalog, and select the "A/B Smartly" destination.
3. Choose which Source should send data to the "A/B Smartly" destination.
4. Go to the [A/B Smartly dashboard](https://segment.absmartly.com/apikey/list), find and copy the "API key" that you created for segment.
Copy link
Contributor

Choose a reason for hiding this comment

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

@gezgblack Is this URL the correct location that users should go to? Or is this an example of what the URL looks like, and the org name should replace segment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's a good question @markzegarelli! I assumed it would be the same across all customers but let me check with them for sure.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hey @markzegarelli John here, filling in for Shamil. I just updated the link with what the partner sent us and they've updated it on their end as well. Thank you as always!

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @abcdefghijohn. Just so I'm clear, are all URLs that point to A/B Smartly unique to the customer's account? The rest of the links in the setup steps point to https://segment.absmartly.com. I assume that we should update the rest of these to. https://your-org-name.absmartly.com and remove the hyperlink, but just want to make sure.

Copy link
Contributor

Choose a reason for hiding this comment

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

@markzegarelli Yes, absolutely, thank you for the catch! I've updated all the links to "your-org-name" now.

Copy link
Contributor

@stayseesong stayseesong left a comment

Choose a reason for hiding this comment

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

@markzegarelli I added some minor edits and didn't touch the links as I saw that you already asked a question about them, and so I just left them as is until you get an answer.

abcdefghijohn and others added 17 commits August 9, 2021 18:05
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
update setup links from segment to yourorg
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
Co-authored-by: stayseesong <83784848+stayseesong@users.noreply.github.com>
@abcdefghijohn
Copy link
Contributor

abcdefghijohn commented Aug 17, 2021

Howdy @markzegarelli , just wondering if everything looks good to go. 😬

@markzegarelli
Copy link
Contributor

markzegarelli commented Aug 18, 2021

@abcdefghijohn Just a few more things for me to clean up. I will do those now.

Thanks!

Copy link
Contributor

@markzegarelli markzegarelli left a comment

Choose a reason for hiding this comment

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

@abcdefghijohn Good to go. I removed the hyper links from the Getting Started steps, as https://your-org-name.absmartly.com is not a valid URL.

Also, I don't see a catalog entry for A/B Smartly, is this something you're waiting on until the docs are ready?

@abcdefghijohn
Copy link
Contributor

@markzegarelli I just created the contents.io page for ab-smartly. I was waiting on it because we're switching over to contentful and just had our training on it yesterday. Thanks for all your help Mark!

@abcdefghijohn abcdefghijohn merged commit e9a7553 into master Aug 25, 2021
@abcdefghijohn abcdefghijohn deleted the abSmartlyDocs branch August 25, 2021 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants