Skip to content
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.

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

Documentation for nextResults and bulk delete #2603

Merged
merged 5 commits into from
Oct 24, 2023

Conversation

lyleschemmerling
Copy link
Contributor

See issue FusionAuth/fusionauth-issues#494

This is the documentation for the search api updates for users and entities for extended pagination using Elasticsearch and an update to add limit to bulk delete API

@spwitt spwitt self-requested a review October 23, 2023 14:54
Copy link
Contributor

@spwitt spwitt left a comment

Choose a reason for hiding this comment

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

Also check out _scim-limits.adoc. It has a reference to the 10,000 limit (with a comma).

@spwitt spwitt self-requested a review October 23, 2023 19:54
Copy link
Contributor

@spwitt spwitt left a comment

Choose a reason for hiding this comment

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

More comma changes.

There is also a reference to the search limitation in _scim-limits.adoc that was mentioned in the issue description doc tasks that should be updated.

site/docs/v1/tech/core-concepts/users.adoc Outdated Show resolved Hide resolved
site/docs/v1/tech/core-concepts/users.adoc Outdated Show resolved Hide resolved
site/docs/v1/tech/core-concepts/users.adoc Outdated Show resolved Hide resolved
site/docs/v1/tech/core-concepts/users.adoc Show resolved Hide resolved
site/docs/v1/tech/core-concepts/users.adoc Outdated Show resolved Hide resolved
@@ -1,6 +1,6 @@
The current SCIM implementation does not support bulk operations using SCIM. Bulk operations on users, such as addition and deletion, are available via link:/docs/v1/tech/apis/[FusionAuth APIs], but are not currently supported via SCIM compatible endpoints. If this functionality is required, please file a https://github.com/fusionauth/fusionauth-issues/issues[GitHub issue] explaining your use case.

The list operation is limited to 10,000 users if you are using the Elasticsearch search engine. This https://github.com/FusionAuth/fusionauth-issues/issues/494[GitHub issue] discusses this limitation.
The list operation is limited to 10,000 users if you are using the Elasticsearch search engine. See link::/docs/v1/tech/reference/limitations#user-searches[Limitations] for more information
Copy link
Contributor Author

Choose a reason for hiding this comment

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

still a potential issue in scim so just changed the reference from the issue to the limitations page

@lyleschemmerling lyleschemmerling merged commit 2dd1a83 into release/1.48.0 Oct 24, 2023
3 checks passed
@lyleschemmerling lyleschemmerling deleted the lyle/release/1.48.0 branch October 24, 2023 15:52
lyleschemmerling added a commit that referenced this pull request Oct 26, 2023
* add docs for webhook signature (#2591)

* add docs for webhook signatures

* Apply suggestions from code review

Co-authored-by: Spencer Witt <[email protected]>

* PR feedback cleanup.  screenshot touchup

* fix api request fields for webhook.signatureConfiguration

* move webhook code pieces to separate repo and include them

* fix tags

* Apply suggestions from code review

Co-authored-by: Spencer Witt <[email protected]>

* minor fixes, link to key master guide

* update screenshot of signing key

---------

Co-authored-by: Spencer Witt <[email protected]>

* Document specifying webhook certificates from Key Master (#2594)

* docs for webhook test reset

* Documentation for `nextResults` and bulk delete (#2603)

* next results documentation

* copy

* pr feedback

* commas

* period

* sync with master

---------

Co-authored-by: Brent Halsey <[email protected]>
Co-authored-by: Spencer Witt <[email protected]>
Co-authored-by: Andy Pai <[email protected]>
Co-authored-by: Daniel DeGroff <[email protected]>
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