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

Add Get all environments task #96

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pegail
Copy link

@pegail pegail commented Jun 11, 2018

Take a path as input, and return all XLD environments available in this path (ex : Environments/MyEnvsFolder/)
The "Get Child" checkbox will allow to search for environments recursively in that path
The "Strip Environments" checkbox will cut the "Environments/" string in the paths returned, so that you are able to directly use one of the path returned as an input to a XLD deploy task.

BERTHIER Clement (IT-CE) added 3 commits June 11, 2018 10:26
Take a path as input, and return all XLD environments available in this path (ex : Environments/MyEnvsFolder/)
The "Get Child" checkbox will allow to search for environments recursively in that path
Possibility to browse for package recursively
Add descriptions to parameters for clarity
@hierynomus
Copy link
Contributor

Never ever use pagesize -1! Please fix this correctly

@pegail
Copy link
Author

pegail commented Jun 15, 2018

Hello,
By pagesize -1 do you mean not using resultsPerPage=-1 in my queries ?
If so no problem, i was just using it because it is already used in existing functions.

@hierynomus
Copy link
Contributor

Yes, and that is something that we need to abolish for all calls. resultsPerPage=-1 means that all resultst will be fetched in a non-paginated mode. For large-scale installations this doesn't work. That's why no new code should ever use that. And wherever you come across it, please fix it.

Remove the resultsPerPage=-1 in all queries in XLDeployClient
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