Skip to content

Add warning that hashing detection applies only to action-destinations #7776

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
Jul 24, 2025

Conversation

harsh-joshi99
Copy link
Contributor

Proposed changes

Slack Thread: https://twilio.slack.com/archives/CC97A542H/p1752733655757679

The previous documentation on pre-hash detection was causing confusion about whether this feature also applies to Classic Destinations.

This PR updates the wording to make it explicit that automatic hash detection is supported only for Action Destinations.

Merge timing

  • ASAP once approved

@harsh-joshi99 harsh-joshi99 requested a review from a team as a code owner July 18, 2025 06:27
@harsh-joshi99 harsh-joshi99 requested review from pwseg and Copilot and removed request for pwseg July 18, 2025 06:27
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds clarification to the documentation about hash detection functionality, specifying that it only applies to Action Destinations and not Classic Destinations. This change addresses user confusion about the scope of the automatic hash detection feature.

  • Adds a warning callout to explicitly state that hashing detection is limited to Action Destinations
  • Improves documentation clarity to prevent misunderstandings about feature availability

@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Jul 24, 2025
@pwseg pwseg merged commit 44a5964 into develop Jul 24, 2025
4 checks passed
@pwseg pwseg deleted the STARTCONN-6052/smart-hashing branch July 24, 2025 06:59
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.

2 participants