Skip to content

Conversation

tysonmote
Copy link
Contributor

Proposed changes

I updated the documented Source Function limits to match our actual limits.

Merge timing

ASAP once approved.

Related issues (optional)

N/A

@netlify
Copy link

netlify bot commented May 17, 2021

Deploy preview for segment-docs ready!

Built with commit 1abd91b

https://deploy-preview-1602--segment-docs.netlify.app

Copy link
Contributor

@sanscontext sanscontext left a comment

Choose a reason for hiding this comment

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

Quick copy edit request :)

tysonmote and others added 3 commits May 17, 2021 16:47
Co-authored-by: LRubin <sanscontext@users.noreply.github.com>
I don't know if this other copy is referenced anywhere but I'm keeping it up-to-date just to be consistent.
@tysonmote
Copy link
Contributor Author

@sanscontext Done! I copied over the text to a page that seems to be a copy of this documentation page in case its referenced somewhere but maybe you have better insight into whether we should delete the duplicate page or not?

@sanscontext
Copy link
Contributor

@tysonmote awesome, thanks! That second file is ??? - that's the old version of the doc at /connections/functions/source-functions/ that shouldn't still be there - it was deleted when we changed the repo structure. I don't know how it came back. 🧟

@sanscontext
Copy link
Contributor

Beautimus! Approved!

@sanscontext sanscontext merged commit 63394a9 into master May 18, 2021
@sanscontext sanscontext deleted the update-source-functions-limit branch May 18, 2021 17:33
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.

3 participants