- Notifications
You must be signed in to change notification settings - Fork396
fixes for batch 5.3#8096
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
fixes for batch 5.3#8096
Uh oh!
There was an error while loading.Please reload this page.
Conversation
netlifybot commentedOct 26, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview forsegment-docs ready!
To edit notification comments on pull requests, go to yourNetlify project configuration. |
| >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. | ||
DeflaimunOct 26, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
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, to prevent markdown list order to reset (previously the html was used to prevent that from happening)
| 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. |
DeflaimunOct 26, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
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
| <divclass="content"> | ||
| <p class="header">Real-time to batch destination sync frequency</p> | ||
| <p>Real-time audience syncsto {{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> |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks great!
3579b8e intodevelopUh oh!
There was an error while loading.Please reload this page.
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
* fixes for batch 5.3* remove liquid syntax and keep the note generic* fix numbering
Uh oh!
There was an error while loading.Please reload this page.
Proposed changes
Required changes for batch 5.3. Fixing HTML elements and target blank links
Merge timing
Auto-merge is ENABLED.
Related issues (optional)