Skip to content

Conversation

jkaho
Copy link
Contributor

@jkaho jkaho commented Jul 25, 2025

Proposed changes

Update docs to reflect changes to algolia-insights action destination.

Also include some other unrelated updates to outdated text / missing info.

Merge timing

Publish after action destination change is merged.

Related issues (optional)

segmentio/action-destinations#3104

Copy link
Contributor

@sade-wusi sade-wusi left a comment

Choose a reason for hiding this comment

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

Added {:target="_blank"} for external links

@sade-wusi sade-wusi added the content-update updates to content that are not new features, includes grammar fixes, added notes label Jul 25, 2025
sade-wusi
sade-wusi previously approved these changes Jul 28, 2025
@jkaho
Copy link
Contributor Author

jkaho commented Aug 14, 2025

@sade-wusi I've pushed an update based on a suggestion from the related PR: segmentio/action-destinations#3104 (comment) 🙂

@jkaho jkaho requested a review from sade-wusi August 14, 2025 08:09
@joe-ayoub-segment
Copy link
Contributor

hi @jkaho I'm good with this to be deployed asap.

@joe-ayoub-segment
Copy link
Contributor

hi @jkaho looks like the docs team would like a change to the PR. Can you please review?

Copy link
Contributor

@sade-wusi sade-wusi left a comment

Choose a reason for hiding this comment

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

Lgtm!

@sade-wusi sade-wusi merged commit 1e61450 into segmentio:develop Aug 26, 2025
4 checks passed
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
content-update updates to content that are not new features, includes grammar fixes, added notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants