Skip to content

Conversation

@Deflaimun
Copy link
Contributor

@Deflaimun Deflaimun commented Oct 26, 2025

Proposed changes

Required changes for batch 5.3. Fixing HTML elements and target blank links

Merge timing

Auto-merge is ENABLED.

Related issues (optional)

@Deflaimun Deflaimun requested a review from a team as a code owner October 26, 2025 12:50
@Deflaimun Deflaimun requested a review from sade-wusi October 26, 2025 12:50
@netlify
Copy link

netlify bot commented Oct 26, 2025

Deploy Preview for segment-docs ready!

Name Link
🔨 Latest commit 1c018d1
🔍 Latest deploy log https://app.netlify.com/projects/segment-docs/deploys/68fe218581c5ad0008093417
😎 Deploy Preview https://deploy-preview-8096--segment-docs.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 project configuration.

Comment on lines +46 to +48
> warning "Added or updated records is the only supported additive sync mode"
> Selecting any other sync mode might lead to sync failures with the Facebook Custom Audiences (Actions) destination.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved the warning to top, so the markdown list order doesn't happen

Save your changes and proceed to [Configure the Send Action](#configure-the-send-action).
</li>
</ol>
1. On the Settings tab, enter values into the **Client ID**, **Brokers** and **Authentication Mechanism** setting fields.
Copy link
Contributor Author

@Deflaimun Deflaimun Oct 26, 2025

Choose a reason for hiding this comment

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

The original author intended the whole page to be a set of steps.

However, the lists are introduced at each paragraph. Reseting the number here still makes sense as they're a different set of steps, even though as a whole its a continuum action

<div class="content">
<p class="header">Real-time to batch destination sync frequency</p>
<p>Real-time audience syncs to {{page.title| replace: "Destination", " "}} may take six or more hours for the initial sync to complete. Upon completion, a sync frequency of two to three hours is expected.</p>
<p>Real-time audience syncs may take six or more hours for the initial sync to complete. Upon completion, a sync frequency of two to three hours is expected.</p>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Some usages showed

Real-time audience syncs to Display and Video 360 (Actions) ...

Real-time audience syncs to Facebook Custom Audiences ... 

Real-time audience syncs to Iterable Lists (Actions) ...

Modifying the snippet to make it generic, doesn't remove context. If you're inside a page already, then it's obvious that this statement applies to this subject.

@Deflaimun Deflaimun enabled auto-merge (squash) October 26, 2025 13:32
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