Skip to content

Recombee AI destination docs #1674

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 29 commits into from
Jun 29, 2021
Merged

Recombee AI destination docs #1674

merged 29 commits into from
Jun 29, 2021

Conversation

gezgblack
Copy link
Contributor

Proposed changes

These are the destination docs for Recombee AI.

I've disabled the settings section from being pulled in from Partner Portal because it lacks a lot of details that the team at Recombee wanted to add including the use of markup. I've added their elaborate Settings section at the bottom of this doc, the same way the usual Settings section gets added.

Please do not merge yet. I will add the updated destinations.yml file merge by myself once approved.

Merge timing

Not urgent yet.

@netlify
Copy link

netlify bot commented Jun 17, 2021

✔️ Deploy Preview for segment-docs ready!

🔨 Explore the source changes: c8de4ab

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

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

@sanscontext
Copy link
Contributor

@gezgblack I have a bunch of edits, sorry this seems like a lot. I appreciate how thorough the partner team is being here. :)

Question about the Settings fields - the fields we import from the Partner Portal/ConfigAPI are in markdown. The reason we pull them in is because for many destinations they're quite detailed, so I'd consider actually putting this text into those fields if possible. Happy to discuss in case I'm missing a good reason to split them out here. :)

gezgblack and others added 2 commits June 18, 2021 09:23
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
gezgblack and others added 14 commits June 18, 2021 09:24
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
@gezgblack
Copy link
Contributor Author

@gezgblack I have a bunch of edits, sorry this seems like a lot. I appreciate how thorough the partner team is being here. :)

Question about the Settings fields - the fields we import from the Partner Portal/ConfigAPI are in markdown. The reason we pull them in is because for many destinations they're quite detailed, so I'd consider actually putting this text into those fields if possible. Happy to discuss in case I'm missing a good reason to split them out here. :)

Hey @sanscontext, the confusion about the settings had arisen because the settings section that the customer fills out in their Developer Center doesn't seem like it includes markdown. Here's a look at what that settings section looks like for the partners:
image
As you can see, the description field looks basic and from their experimentations, didn't support markdown.

You mentioned that Partner-Portal that we have access to supports markdown. Here's the issue that I see. This is their Partner-Portal settings section: https://app.segment.com/partner-portal/integration/recombee-ai/settings. The data that you see there like the "description" property shows up in the destination settings when they try to set up the destination. I feel like if we put all of that settings description into their Partner Portal settings it will look cluttered in the destination settings. The other question is, should I manually enter those long additional settings into their Partner-Portal instance? Should I add them on top of their existing descriptions or remove their current descriptions from Partner-Portal and override them with the long list?

@sanscontext
Copy link
Contributor

sanscontext commented Jun 21, 2021

Hey @gezgblack if you look at the settings for, for example Braze, or Adobe Analytics, their settings descriptions do have markdown included. If this option isn't showing up correctly in the Partner Portal that's maybe a question for whoever is currently maintaining that interface? I'm not sure who that'd be though 🤔

Stylistically, while we'd like the descriptions to be shorter and less-cluttered, prior art is to put as much context as needed into the description for the sake of the user. Given that all the docs research I've ever seen says that people will do anything they can to not navigate away from a page to look at docs, I think this is still our best option. Happy to have a discussion with you and whoever else?

@gezgblack
Copy link
Contributor Author

Hey @gezgblack if you look at the settings for, for example Braze, or Adobe Analytics, their settings descriptions do have markdown included. If this option isn't showing up correctly in the Partner Portal that's maybe a question for whoever is currently maintaining that interface? I'm not sure who that'd be though 🤔

Stylistically, while we'd like the descriptions to be shorter and less-cluttered, prior art is to put as much context as needed into the description for the sake of the user. Given that all the docs research I've ever seen says that people will do anything they can to not navigate away from a page to look at docs, I think this is still our best option. Happy to have a discussion with you and whoever else?

The issue is that the markdown is not available in the Developer Center Settings Builder. It looks like it is available in the Partner Portal. The partner doesn't have access to the Partner Portal, they only have access to the Developer Center Settings Builder. With that in mind, should I manually copy-paste their setting into their Partner Portal settings section?

@sanscontext
Copy link
Contributor

Oh wow! I didn't realize we were maintaining two different UIs for this. Let's discuss this on the channel so I don't miss anything - there might have been a good reason for them not to allow markdown.

@gezgblack
Copy link
Contributor Author

I believe this one should be good to go now @sanscontext. Take a look and let me know if you're happy enough with this.

⚠️⚠️⚠️ Please don't merge, just looking for approval ⚠️⚠️⚠️

I'll merge myself once I add the updated destinations.yml file.

@sanscontext
Copy link
Contributor

Hey @gezgblack let me reformat the settings just a bit, and then I'll approve. :)

Copy link
Contributor

@sanscontext sanscontext left a comment

Choose a reason for hiding this comment

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

Approved! Thanks so much for your patience with the settings stuff! Merge when you're ready. :)

@gezgblack gezgblack merged commit 6d4a826 into master Jun 29, 2021
@gezgblack gezgblack deleted the destination/Recombee branch June 29, 2021 10:58
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.

2 participants