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 — docs(salesforce): correct the setup steps that would strand an admin by waleedlatif1 · Pull Request #6526 · simstudioai/sim

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to #6508. Verified the Salesforce integration-user guide against Salesforce's current UI and first-party docs before we point a customer at it. Most of it holds; four things do not, and one of them is the likeliest way a JWT setup fails.

  • The Client Credentials step pointed at the wrong screen. It told admins to check Enable Client Credentials Flow under OAuth Policies. On an External Client App that checkbox is under Edit Settings → OAuth Settings; the Policies page holds only the Run As picker. Anyone following it literally hunts for a control that isn't there. The FAQ answer inherited the same conflation.
  • Pre-authorizing the app must go through the profile or a second permission set. A permission set backed by the Salesforce API Integration license cannot hold an Assigned Connected Apps section at all (KB 001453686), so the app can never be assigned from the same permission set that grants object access — producing exactly the user hasn't approved this consumer failure that step exists to prevent.
  • RSA ≥2048 only. An ECDSA key is silently rejected, and the uploaded certificate must stay under 4 KB.
  • The JWT toggle doesn't appear until Enable OAuth is on, and the upload control is Upload Files.

Also scopes the capability promise for the API-only license, so we don't advertise something untested:

  • SOQL and CRUD on standard objects — supported.
  • Reports and dashboards — genuinely unverified in either direction from first-party docs. Flagged as needing a sandbox test rather than asserted.
  • Apex Class Access is a permission this license cannot hold, so Tooling API calls touching ApexClass will fail. Custom field/object management is unaffected.

Verified as accurate and left alone: External Client App is the right object, Connected App creation is disabled org-wide since Spring '26, the openssl command and a 10-year self-signed cert are fine, the My Domain formats are current post-Enhanced-Domains, and "Admin approved users are pre-authorized" is the verbatim current label.

Type of Change

  • Documentation

Testing

fumadocs-mdx + tsc --noEmit clean, biome clean, MDX component nesting balanced and the FAQ block still parses.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Sorry, something went wrong.

Uh oh!

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

greptile-apps[bot] reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@waleedlatif1
docs(salesforce): correct the setup steps that would strand an admin
2cc67c4
Verified the guide against Salesforce's current UI and docs. Most of it holds;
these do not:

- The Client Credentials step told admins to check "Enable Client Credentials
  Flow" under OAuth Policies. On an External Client App that checkbox is under
  Edit Settings → OAuth Settings; the Policies page holds only the Run As
  picker, so anyone following it literally hunts for a control that is not on
  the screen. The FAQ answer inherited the same conflation.
- Pre-authorizing the app must go through the profile or a SECOND permission
  set. A permission set backed by the Salesforce API Integration license cannot
  hold an Assigned Connected Apps section at all, so the app can never be
  assigned from the same permission set that grants object access — which
  produces exactly the "user hasn't approved this consumer" failure that step
  exists to prevent. This is the likeliest way a JWT setup fails.
- Salesforce requires an RSA key of at least 2048 bits; an ECDSA key is
  silently rejected, and the certificate must stay under 4 KB.
- The JWT toggle does not appear until Enable OAuth is on, and the control is
  "Upload Files".

Also scopes the capability promise for the API-only license: SOQL and CRUD on
standard objects are supported, reports and dashboards are genuinely unverified
in either direction, and Apex Class Access is a permission this license cannot
hold, so Tooling API calls touching ApexClass will fail.
@vercel

vercel Bot commented Aug 11, 2026 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 11, 2026 3:30am

Request Review

All reactions

Sorry, something went wrong.

Uh oh!

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

@cursor

cursor Bot commented Aug 11, 2026 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown

PR Summary

Low Risk
Documentation-only changes to the Salesforce integration guide; no application code, auth, or data paths modified.

Overview
Updates the Salesforce integration user guide so admins follow the right Salesforce UI paths and avoid common JWT / Client Credentials failures.

Client credentials: Step 2 now points to Edit Settings → API (Enable OAuth Settings) for Enable OAuth. Step 3 clarifies that Enable Client Credentials Flow lives in OAuth Settings (step 2), while Policies only sets Run As—fixing instructions that sent people hunting for a checkbox on the wrong screen.

JWT bearer: Adds warnings for RSA ≥2048 keys and 4 KB cert limit; notes Enable OAuth must be on before the JWT toggle and to use Upload Files for server.crt. Adds a critical callout that app pre-authorization must use the profile or a second permission set without the Salesforce API Integration license—assigning from the object-access permission set cannot work and causes user hasn't approved this consumer.

My Domain: Corrects Developer Edition host to yourorg.develop.my.salesforce.com (with -dev-ed only when Salesforce generated the name); FAQ aligned.

API-only license: New callout limits the promise—SOQL/CRUD on standard objects OK; reports/dashboards unverified; Apex Class Access blocked on Tooling API ApexClass calls.

Reviewed by Cursor Bugbot for commit fcf8167. Configure here.

All reactions

Sorry, something went wrong.

Uh oh!

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

@greptile-apps

greptile-apps Bot commented Aug 11, 2026 •
edited
Loading

Uh oh!

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

Copy link
Copy Markdown
Contributor

Greptile Summary

The documentation now corrects Salesforce External Client App setup instructions and clarifies JWT, permission-set, API-only license, and My Domain requirements.

  • Separates Client Credentials enablement from the Run As policy configuration.
  • Documents RSA certificate and JWT pre-authorization constraints.
  • Qualifies API-only license support for reports, dashboards, and Apex.
  • Makes Developer Edition hostname guidance consistent between setup and FAQ sections.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/docs/content/docs/en/integrations/salesforce-service-account.mdx Updates the Salesforce service-account guide, including a complete fix for the previously conflicting Developer Edition hostname instructions.

Reviews (2): Last reviewed commit: "docs(salesforce): align the Developer Ed..." | Re-trigger Greptile

All reactions

Sorry, something went wrong.

Uh oh!

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

greptile-apps[bot]
greptile-apps Bot reviewed Aug 11, 2026
View reviewed changes
Comment thread apps/docs/content/docs/en/integrations/salesforce-service-account.mdx Resolved

Uh oh!

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

@waleedlatif1
docs(salesforce): align the Developer Edition host in the FAQ with th…
fcf8167
…e setup section

The setup section was corrected to make the `-dev-ed` suffix conditional, but
the FAQ still presented it as mandatory — so an admin whose Developer Edition
domain lacks the generated suffix would read two contradictory formats on the
same page and validate against a host that does not exist.
@waleedlatif1

waleedlatif1 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

@greptile

greptile-apps[bot] 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.

@waleedlatif1

waleedlatif1 commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

@cursor review

cursor[bot] 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.

cursor[bot]
cursor Bot reviewed Aug 11, 2026
View reviewed changes

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit fcf8167. Configure here.

Sorry, something went wrong.

Uh oh!

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

All reactions
@vercel
vercel Bot deployed to Preview August 11, 2026 03:30
View deployment
@waleedlatif1
waleedlatif1 merged commit daac4f3 into staging Aug 11, 2026
6 checks passed

Uh oh!

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

@waleedlatif1
waleedlatif1 deleted the salesforce-docs-accuracy branch August 11, 2026 03:33
This was referenced Aug 11, 2026
v0.8.0: new workflow UI, v2 API, salesforce JWT, provenance improvements, incidentio tools, block retries #6533
Closed
v0.8.0: desktop app, workflow UI, v2 endpoints #6606
Merged

This branch was successfully deployed

1 active deployment
Preview — fcf81678 Deployed Aug 11, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@cursor cursor[bot] cursor[bot] left review comments

Uh oh!

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

Assignees

No one assigned

Labels

None yet

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.

1 participant

@waleedlatif1
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 01402503 — 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.