Skip to content

Add Node.js tasks#362

Merged
crandmck merged 3 commits intomainfrom
js-tasks
May 7, 2026
Merged

Add Node.js tasks#362
crandmck merged 3 commits intomainfrom
js-tasks

Conversation

@crandmck
Copy link
Copy Markdown
Collaborator

@crandmck crandmck commented Apr 28, 2026

Changes in this pull request

New pages in Working with the SDK in Node.js.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 28, 2026

Deploy Preview for cai-open-source ready!

Name Link
🔨 Latest commit f6e0f60
🔍 Latest deploy log https://app.netlify.com/projects/cai-open-source/deploys/69fcb113ea544600085a68b5
😎 Deploy Preview https://deploy-preview-362--cai-open-source.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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

@github-actions github-actions Bot temporarily deployed to pull request April 28, 2026 22:03 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 6, 2026 19:52 Inactive
@crandmck
Copy link
Copy Markdown
Collaborator Author

crandmck commented May 6, 2026

@timmurphydev Made the specific changes we discussed, plus changed example code to JS instead of TS:

  • Types removed: Parameter and return types, and annotations like { buffer: Buffer | null }, are plain JS (const dest = { buffer: null }, async (data) => { ... }, untyped function params).
  • _node-read.md: Comment read-manifest.ts → read-manifest.js, usage ts-node read-manifest.ts → node read-manifest.js (matches ESM-style examples with import).
  • _node-settings.md: Text now says “settings fragments” instead of “typed fragments” so it doesn’t imply TypeScript.

@github-actions github-actions Bot temporarily deployed to pull request May 7, 2026 15:38 Inactive
Copy link
Copy Markdown
Collaborator

@timmurphydev timmurphydev left a comment

Choose a reason for hiding this comment

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

It looks like everything we discussed has been updated. This looks good to me, Rand!

@crandmck crandmck merged commit 55dbcd0 into main May 7, 2026
7 checks passed
@crandmck crandmck deleted the js-tasks branch May 7, 2026 20:17
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.

2 participants