GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add support for external-callout by diksha-sf · Pull Request #134 · forcedotcom/datacloud-customcode-python-sdk

Conversation

@diksha-sf

@diksha-sf diksha-sf commented Jul 30, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Validated using pre-release version: v6.1.0.dev3

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
jcatt-sf and others added 14 commits July 1, 2026 10:54
@jcatt-sf
Add streaming delta read/write methods to the SDK
b470208
@jcatt-sf
documentation fixes and write mode removal
5837e4c
@jcatt-sf
Merge pull request forcedotcom#123 from forcedotcom/streaming-delta-s…
6df6d83 
…dk-methods

Add streaming delta read/write methods to the SDK
@jcatt-sf
rely on config for name source
3da41b6
@jcatt-sf
separate streaming client
2c3d285
@jcatt-sf
streaming source from config
5baa65a
@jcatt-sf
streaming source fix
f6deec0
@jcatt-sf
lint
2cb85d3
@jcatt-sf
Merge pull request forcedotcom#124 from forcedotcom/remove-streaming-…
1022658 
…delta-source-name

rely on config for name source
@jcatt-sf
Support creating and deploying streaming script code packages
4c00881
@jcatt-sf
lint
5738e80
@jcatt-sf
Merge pull request forcedotcom#132 from forcedotcom/Support-creating-…
e0603af 
…and-deploying-streaming-script-code-packages

Support creating and deploying streaming script code packages
@sbyrne-sf
Merge pull request forcedotcom#133 from forcedotcom/main
5317119 
Sync Develop with Main
@diksha-sf
Add support for external-callout
80261cb
@salesforce-cla salesforce-cla Bot added the cla:signed label Jul 30, 2026
@diksha-sf
Fix lint errors
10bb3d5
@atulikumwenayo

atulikumwenayo commented Jul 30, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator
  1. I think we should expect any string format for the input, and any string format for the server response, we shouldn't assume what they are: json, text, xml, etc. This also means we shouldn't have a default value for Content-Type header
  2. Exposing only one method (request) and removing callout_json would suffice.
  3. It should be status_code not http_status_code
  4. Should we let the user control the response timeout, maybe with a hard max that we don't allow them to exceed?
diksha-sf reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
diksha-sf added 2 commits August 3, 2026 10:31
@diksha-sf
Address review comments
756fc43
@diksha-sf
Fix build error and add example
1f2096d
@diksha-sf diksha-sf mentioned this pull request Aug 3, 2026
Add support for external-callout #135
Closed
@diksha-sf
diksha-sf temporarily deployed to pypi August 3, 2026 06:54 — with GitHub Actions Inactive
@diksha-sf

diksha-sf commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

@atulikumwenayo addressed the PR review comments 1-3. For 4, I synced with @sarasavilli-sf, this would need more work. Created WI - W-23683713
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
diksha-sf added 2 commits August 3, 2026 16:21
@diksha-sf
Fix error on deploy
bcfa03d
@diksha-sf
Version bump in mock file
fa645b5
@diksha-sf
diksha-sf requested a review from atulikumwenayo August 3, 2026 11:10
@diksha-sf
diksha-sf temporarily deployed to pypi August 3, 2026 11:17 — with GitHub Actions Inactive
@diksha-sf
diksha-sf requested a review from ritaagarwala-sf August 3, 2026 13:48
diksha-sf added 3 commits August 3, 2026 20:45
@diksha-sf
merge with develop
3ea49fa
@diksha-sf
revert change to mock server
75ad623
@diksha-sf
Add example for script
12eabe1
diksha-sf added 2 commits August 10, 2026 20:24
@diksha-sf
Add support for timeout header
2c9e319
@diksha-sf
Merge branch 'forcedotcom:main' into main
fb22a94

This branch was previously deployed

1 inactive (outdated) deployment
pypi — fa645b5e Deployed Aug 3, 2026 by diksha-sf via publish #72
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@atulikumwenayo atulikumwenayo Awaiting requested review from atulikumwenayo
@ritaagarwala-sf ritaagarwala-sf Awaiting requested review from ritaagarwala-sf

At least 1 approving review is required to merge this pull request.

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

cla:signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@diksha-sf @atulikumwenayo @jcatt-sf @sbyrne-sf
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.