Skip to content

added more documentation for SaaS Connectivity - #1222

Open
philip-ellis-sp wants to merge 2 commits into
mainfrom
feature/DEVREL-3542
Open

philip-ellis-sp wants to merge 2 commits into
mainfrom
feature/DEVREL-3542

Conversation

@philip-ellis-sp

@philip-ellis-sp philip-ellis-sp commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Added documentation for new SDK commands SSF is documented but hidden as draft since it is used for internal purposes only.

@github-actions

github-actions Bot commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

🌎🌎🌎 Visit the preview URL for this PR HERE
built from commit d417254

@dibyaranjan-pradhan-sp

Copy link
Copy Markdown

It looks good.

}
```

SailPoint Identity Security sends `input.datasetIds` with the datasets to discover. SailPoint Identity Security can also send `additionalParameters` to pass extra data to some SailPoint connectors, for example the location of a file to read. Your connector does not need to use it.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are datasetId and datasetIds the same field? This line says SailPoint Identity Security sends input.datasetIds, but the input example shows datasetId and the code reads input.datasetId.

From what I can tell, the platform sends datasetIds (a list), and with stdApplicationDiscoveryListWithDataset the SDK calls your handler once per ID, with that single ID in input.datasetId. If that's right, I'd spell it out here, just in case!


### Declare datasets in the connector spec

Add a `datasets` object to `connector-spec.json`. Each key is a dataset ID. Set `type` to `std:application_discovery`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you confirm std:application_discovery (with the underscore) is the correct dataset type? The command has a hyphen instead of an underscore. If it is, I would add a short note that it differs from the command name, since it will be easy to copy incorrectly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes this is actually correct.

This branch has not been deployed

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants