Skip to content

Conversation

asimms41
Copy link
Contributor

@asimms41 asimms41 commented Mar 5, 2025

Description

Resolves DOC-1065
Review deadline: 10th March

Interim solution: Adding a message to alert users that they need to enable connectivity.

This pull request introduces changes to improve the documentation for external connectivity in various input and output components. The key changes include adding partials for external connectivity instructions and including these partials in the relevant documentation files.

Documentation improvements:

Addition of external connectivity partials:

Page previews

Inputs

Outputs

Checks

  • New feature
  • Content gap
  • Support Follow-up
  • Small fix (typos, links, copyedits, etc)

Copy link

netlify bot commented Mar 5, 2025

Deploy Preview for redpanda-connect ready!

Name Link
🔨 Latest commit faaf387
🔍 Latest deploy log https://app.netlify.com/sites/redpanda-connect/deploys/67c8641feca6380008e6e831
😎 Deploy Preview https://deploy-preview-185--redpanda-connect.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -0,0 +1,15 @@
ifndef::env-cloud[]

Copy link
Contributor Author

@asimms41 asimms41 Mar 5, 2025

Choose a reason for hiding this comment

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

I've added this message to just the timeplus input for now. See page previews for cloud and non-cloud versions. I skimmed through the list but wasn't entirely sure which connectors it might apply to, it didn't seem relevant in many cases. Any guidance appreciated: https://github.com/redpanda-data/connect/blob/main/internal/plugins/info.csv

@@ -0,0 +1,15 @@
ifndef::env-cloud[]

== Enable connectivity to cloud-based data sinks
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added this message to a small set of outputs for now. Again, any guidance on which other data sinks this should be added to would be appreciated.

@asimms41 asimms41 requested review from rockwotj and supahcraig March 5, 2025 15:18
@rockwotj rockwotj requested a review from jrkinley March 7, 2025 09:25
@rockwotj rockwotj removed their request for review April 26, 2025 23:51
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.

1 participant