-
Notifications
You must be signed in to change notification settings - Fork 367
Destination function batching updates [DOC-527] #3115
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
Conversation
✅ Deploy Preview for segment-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
LGTM. Just curious about the decision to comment images vs remove them from the doc.
|
||
When you click **Build**, a code editor appears. Use the editor to write the code for your function, configure settings, and test the function's behavior. | ||
|
||
 | ||
<!---  ---> |
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.
@forstisabella Is there a reason to comment this out vs remove the image?
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.
@markzegarelli That's my mistake: I commented out the images to test if they were integral to the docs (this one wasn't.) I'll go back through and make sure they're all removed!
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
Proposed changes
Merge timing
On June 30, 2022
Related issues (optional)
DOC-527