Skip to content

Allow to set search_engine version for Users API#140

Merged
lbalmaceda merged 1 commit intomasterfrom
search-v3
Jul 12, 2018
Merged

Allow to set search_engine version for Users API#140
lbalmaceda merged 1 commit intomasterfrom
search-v3

Conversation

@lbalmaceda
Copy link
Copy Markdown
Contributor

Since the search_engine v2 is no longer allowed for new tenants and it's going to be shut down on November 13th this year, this PR adds a setter to specify a different value than the default one.

Fixes #133

@lbalmaceda lbalmaceda added this to the v1-Next milestone Jul 12, 2018
* Filter by a query
*
* @param query the query expression using the following syntax https://auth0.com/docs/api/management/v2/query-string-syntax.
* @param query the query expression to use
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.

this filter applies both for Users and Logs. No need to mention the User syntax here.

*/
public UserFilter() {
parameters.put("search_engine", "v2");
withSearchEngine("v2");
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.

Avoids a breaking change.

Copy link
Copy Markdown
Contributor

@joshcanhelp joshcanhelp left a comment

Choose a reason for hiding this comment

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

LGTM

@lbalmaceda lbalmaceda merged commit 7a56911 into master Jul 12, 2018
@lbalmaceda lbalmaceda deleted the search-v3 branch July 12, 2018 21:02
@lbalmaceda lbalmaceda changed the title Allow to set search_engine version Allow to set search_engine version for Users API Jul 13, 2018
@lbalmaceda lbalmaceda modified the milestones: v1-Next, 1.8.0 Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants