Skip to content

remove snippet #5157

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 1 commit into from
Aug 10, 2023
Merged

remove snippet #5157

merged 1 commit into from
Aug 10, 2023

Conversation

tcgilbert
Copy link
Contributor

Proposed changes

removing irrelevant snippet from documentation

Merge timing

asap

@tcgilbert tcgilbert requested a review from a team as a code owner August 9, 2023 16:01
@tcgilbert tcgilbert requested review from stayseesong and removed request for a team August 9, 2023 16:01
@@ -43,10 +43,6 @@ This destination is maintained by YOURINTEGRATION. For any issues with the desti

## Getting Started

> Do not remove this line. It will auto-populate the following information for your integration:

{% include content/connection-modes.md %}
Copy link
Contributor

Choose a reason for hiding this comment

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

@tcgilbert just curious, how come we're removing the {% include content/connection-modes.md %}?

@stayseesong stayseesong merged commit d0cbb5e into develop Aug 10, 2023
@stayseesong stayseesong deleted the thomas/partners branch August 10, 2023 23:10
@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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants