-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Testing / modifying code examples #2133
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
|
This pull request is being automatically deployed with Vercel (learn more). pipedream-docs-redirect-do-not-edit – ./docs🔍 Inspect: https://vercel.com/pipedreamers/pipedream-docs-redirect-do-not-edit/2KLvwcDBBBsntp4nuPU6LQQjWVYY pipedream-docs – ./docs🔍 Inspect: https://vercel.com/pipedreamers/pipedream-docs/G9eMURYXmAgQZrMYP6SwYrXFakqs |
|
|
||
| [Read the props reference for the full list of options](/components/api/#props). | ||
|
|
||
| ## `async` function declaration |
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.
This section still referenced the old function declaration so I modified to discuss components. lmk if you think this can be improved
| ## `$end` | ||
| ## Ending a workflow early |
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 references to $end still here, just modifying
| export default defineComponent({ | ||
| async run({ steps, $ }) { | ||
| $.flow.exit(); | ||
| return $.flow.exit(); |
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.
I always forget that using $.flow.exit() requires an explicit return. Not a great DX but you can talk to Giao about this one... we've faced some challenges here.
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.
Oh wow I didn't know that. My other examples are broken then. I thought it was the same behavior as $.end
| } | ||
| }, | ||
| async run({ steps, $ }) { | ||
| const web = new WebClient(this.slack.$auths.oauth_access_token) |
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.
Changing references from $auths -> $auth. Data is exposed in the singular object
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.
Thanks, good catch
| const { path, cleanup } = await file(); | ||
| await fs.promises.appendFile(path, Buffer.from("hello, world")) | ||
| await cleanup(); | ||
| }) |
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.
Ah right, legacy from the run
* Changing docs root to prep for docs changes * Committing new vercel.json to handle pathing * Moving to a sub-language navigation and adding Python examples * Adding more python docs about file storage, sending files, etc * Tweaks to python and finishing up bash steps * Basic tmp example for curl * mid edit through the to-do list * Completing review notes * Incorporating dropdown versioning and adding new signifier * Adding fun gradient styling * Playing with a typeface * Removing dead spaces * Fixing mis-configured subsection path * Adding file example * Hack to support internal relative URLs * Good spellchecker * More spell checking * Polish iteration #2.8 * Matching bash format for passing data more closely * Adding a limitation note as well * Adding defineComponent wrapping to main nodejs README and working with files section * Gotta catch'em all ...typos * Adding $.auth and removing checkpoints altogether * Adding $.service.db examples in the state docs * Adding state management docs * Adding 404 checker * Adding spellchecking appeasement & removing loud 404 finder * Dedupe is fine * Docs v2 updates (#2090) Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> * Adding multiworkflow example * Adding Node.js passing data between steps, route addition to fix URL change, typos * python vs Python * Iteration #5 * Spellchecking * Removing obsolete $.flow.end statement in inspector logs * Testing / modifying code examples (#2133) * Forgot we had the async code doc, as well. * Testing code examples. * $auths -> $auth * Migrating HTTP request examples to components * Correcting file examples * Fixing redundant python ENV example * Adding Dylan's Edits * Adding a more concrete file example to fix spellchecker and make it easier to grok the concept * Correcting two links * Fixing broken links * Fixing misnamed to * Redirect docs-v2 to docs/v2 * Empty commit to trigger v2 docs build on vercel * Moving to (.*) format * newline * docs-v2 to docs/v2 * Go bob docs (#2139) * Adding Go examples * Adding Go Lang version config var * Applying review comments * Wordlist and updating vercel * Missing some words * Modifying redirects Co-authored-by: Dylan Pierce <me@dylanjpierce.com> Co-authored-by: Pravin <pravin@pipedream.com>
* Changing docs root to prep for docs changes * Committing new vercel.json to handle pathing * Moving to a sub-language navigation and adding Python examples * Adding more python docs about file storage, sending files, etc * Tweaks to python and finishing up bash steps * Basic tmp example for curl * mid edit through the to-do list * Completing review notes * Incorporating dropdown versioning and adding new signifier * Adding fun gradient styling * Playing with a typeface * Removing dead spaces * Fixing mis-configured subsection path * Adding file example * Hack to support internal relative URLs * Good spellchecker * More spell checking * Polish iteration #2.8 * Matching bash format for passing data more closely * Adding a limitation note as well * Adding defineComponent wrapping to main nodejs README and working with files section * Gotta catch'em all ...typos * Adding $.auth and removing checkpoints altogether * Adding $.service.db examples in the state docs * Adding state management docs * Adding 404 checker * Adding spellchecking appeasement & removing loud 404 finder * Dedupe is fine * Docs v2 updates (#2090) Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> * Adding multiworkflow example * Adding Node.js passing data between steps, route addition to fix URL change, typos * python vs Python * Iteration #5 * Spellchecking * Removing obsolete $.flow.end statement in inspector logs * Testing / modifying code examples (#2133) * Forgot we had the async code doc, as well. * Testing code examples. * $auths -> $auth * Migrating HTTP request examples to components * Correcting file examples * Fixing redundant python ENV example * Adding Dylan's Edits * Adding a more concrete file example to fix spellchecker and make it easier to grok the concept * Correcting two links * Fixing broken links * Fixing misnamed to * Redirect docs-v2 to docs/v2 * Empty commit to trigger v2 docs build on vercel * Moving to (.*) format * newline * docs-v2 to docs/v2 * Go bob docs (#2139) * Adding Go examples * Adding Go Lang version config var * Applying review comments * Wordlist and updating vercel * Missing some words * Adding docs naked * Newline Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> Co-authored-by: Pravin <pravin@pipedream.com>
* Changing docs root to prep for docs changes * Committing new vercel.json to handle pathing * Moving to a sub-language navigation and adding Python examples * Adding more python docs about file storage, sending files, etc * Tweaks to python and finishing up bash steps * Basic tmp example for curl * mid edit through the to-do list * Completing review notes * Incorporating dropdown versioning and adding new signifier * Adding fun gradient styling * Playing with a typeface * Removing dead spaces * Fixing mis-configured subsection path * Adding file example * Hack to support internal relative URLs * Good spellchecker * More spell checking * Polish iteration #2.8 * Matching bash format for passing data more closely * Adding a limitation note as well * Adding defineComponent wrapping to main nodejs README and working with files section * Gotta catch'em all ...typos * Adding $.auth and removing checkpoints altogether * Adding $.service.db examples in the state docs * Adding state management docs * Adding 404 checker * Adding spellchecking appeasement & removing loud 404 finder * Dedupe is fine * Docs v2 updates (#2090) Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> * Adding multiworkflow example * Adding Node.js passing data between steps, route addition to fix URL change, typos * python vs Python * Iteration #5 * Spellchecking * Removing obsolete $.flow.end statement in inspector logs * Testing / modifying code examples (#2133) * Forgot we had the async code doc, as well. * Testing code examples. * $auths -> $auth * Migrating HTTP request examples to components * Correcting file examples * Fixing redundant python ENV example * Adding Dylan's Edits * Adding a more concrete file example to fix spellchecker and make it easier to grok the concept * Correcting two links * Fixing broken links * Fixing misnamed to * Basic outline of the migration guide * WIP - more content * Filling out skeleton more * Accidental commits * Adding http response section * Spellchecking * Simplifying * More editing * Edits * Run through #3 * Adding slack example * Spellchecking * Fixing internal title link scrolling * Adding a few more images and fixing internal links * Adding Pravins updates * Moving migration guide to child nav and adding blurb for v2 only users * Adding testing events reference * Bringing in the video as an embed * Dylans edits v3 * Adding demo poster * Adding Pravins latest edits, fixing youtube Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> Co-authored-by: Pravin <pravin@pipedream.com>
* Changing docs root to prep for docs changes * Committing new vercel.json to handle pathing * Moving to a sub-language navigation and adding Python examples * Adding more python docs about file storage, sending files, etc * Tweaks to python and finishing up bash steps * Basic tmp example for curl * mid edit through the to-do list * Completing review notes * Incorporating dropdown versioning and adding new signifier * Adding fun gradient styling * Playing with a typeface * Removing dead spaces * Fixing mis-configured subsection path * Adding file example * Hack to support internal relative URLs * Good spellchecker * More spell checking * Polish iteration #2.8 * Matching bash format for passing data more closely * Adding a limitation note as well * Adding defineComponent wrapping to main nodejs README and working with files section * Gotta catch'em all ...typos * Adding $.auth and removing checkpoints altogether * Adding $.service.db examples in the state docs * Adding state management docs * Adding 404 checker * Adding spellchecking appeasement & removing loud 404 finder * Dedupe is fine * Docs v2 updates (#2090) Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> * Adding multiworkflow example * Adding Node.js passing data between steps, route addition to fix URL change, typos * python vs Python * Iteration #5 * Spellchecking * Removing obsolete $.flow.end statement in inspector logs * Testing / modifying code examples (#2133) * Forgot we had the async code doc, as well. * Testing code examples. * $auths -> $auth * Migrating HTTP request examples to components * Correcting file examples * Fixing redundant python ENV example * Adding Dylan's Edits * Adding a more concrete file example to fix spellchecker and make it easier to grok the concept * Adding navgrid to v1 vuepress docs [WIP] * Moving navgrid component and starting to separate concepts from the main quickstart * Moving docs into a different hierarchy * Guides concept, splitting out individual guides from the main READMEs * linting * Adding dist dir * Test * One more try * Modifying assets path * Reintroduce the base config * Reverting assets change * Fixing alignment of concepts grid * Fix auto headers on quickstart * Fixing display block of concepts * Adding footer links & better logos for languages * Reverting concepts subdir and adding better layout * Adding account settings and troubleshooting sections * Sidebar config update * Adidng component icon too * Merge branch 'master' into add-navgrid-to-vuepress1 * Grammar fix and more specific fence * Why does VScode hate newlines on vercel.json? * Fixing incorrect sidebar path * Removing redundant base path config in themeConfig * Attempting to fix asset pathing * Replacing $.service.db with this.store docs * Updating stores docs * Adding more specific JSON serailization examples * Fixing a few typos * Another round of edits * Nav fixes * Using new data_store term * Reverting the name change for now * Removing SDK * Replacing images * Removing inferred_body_type which is removed * Moving /docs/triggers * Updated screenshots * Fixing actions update screenshot and description * Removing problem entry * Fixing 404 * Fixing sidebar and adding separate sections * Fixing v-pre key * Adding getting started to top left nav * Fix navgrid overflow on mobile * Removing old commented out docs * Better tmp file size messaging * Removing Guidelinks experiment * Docs nav change out * Don't add overview route * Nav swaps * Fixing SVG path * Fixing merge conflict * Empty commit to trigger v2 docs build on vercel * Removing dead comma * Removing redundant package-lock.json * Removing broken rule * Using cloudinary for the icon * Reorder side nav * Renaming quickstart & adding primary color not button * gco q * Updating data store intro sentence * Splitting the nav * Dylan's edits * Fixing styles too * Fixing sidebar duplicate links and unnecessary dropdowns * More sidebar massaging * Removing dupe workflow links in code * Fixing missing navbar in the sidebar in mobile * Adding `await` to get / set calls to future-proof a potential async implementation * Adding missing pages, moving cold starts out of reference and back under workflows * Adding missing events inspect docs Co-authored-by: Dylan Pierce <me@dylanjpierce.com> Co-authored-by: Pravin <pravin@pipedream.com> Co-authored-by: Danny Roosevelt <danny@pipedream.com>
No description provided.