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.

Sign up for GitHub

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

Jump to bottom

fix: include RecordStream in `jsforce` class #1415

Open

cristiand391 wants to merge 1 commit into 3.0 from cd/record-stream-import

Member

cristiand391 commented May 6, 2024

follow up to docs update, when updating this example in the Advanced sections here:
https://github.com/jsforce/jsforce-website/pull/23/files#diff-a3471f45ee29622bbcef0e1609f20396694da97b82c406a839b8c776fad94424R117

I found the jsforce class in v2 doesn't include RecordStream, v1 included it:
https://github.com/jsforce/jsforce/blob/master/lib/core.js#L14

mostly handy for the filter/map/recordMapStream static methods that return a node transform stream.


          fix: include RecordStream in jsforce class

ead7de1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment