Skip to content

Conversation

gezgblack
Copy link
Contributor

Proposed changes

This is the docs entry for the new Kevel Destination. I have one question and it's about the 2 sections Personas and Settings. I believe these would get added automatically in the past, is that correct? Should I remove these sections from the PR?

Merge timing

Not urgent at the moment.

@netlify
Copy link

netlify bot commented Apr 7, 2021

Deploy preview for segment-docs ready!

Built with commit f07117f

https://deploy-preview-1503--segment-docs.netlify.app

@markzegarelli
Copy link
Contributor

Hi @gezgblack. Yes, the Personas and Settings sections will be populated automatically. However, this will not happen until the destination moves out of Private Beta, and into either Public Beta or GA. Do you have any information about the timing of this happening?

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.

A few small edits, and one question which you might want to get a screenshot from the developer for?

gezgblack and others added 6 commits April 8, 2021 10:28
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

Hi @gezgblack. Yes, the Personas and Settings sections will be populated automatically. However, this will not happen until the destination moves out of Private Beta, and into either Public Beta or GA. Do you have any information about the timing of this happening?

I'll be switching them to Public and adding the updated make catalog file in this PR as soon as you guys 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.

I fixed my own typo. Now approved!

@gezgblack gezgblack merged commit b9cae3e into master Apr 9, 2021
@gezgblack gezgblack deleted the destination/kevel branch April 9, 2021 15:02
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.

3 participants