-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: Introduce MaximumFollowNextPageLink option for usage with Get-AzLogicAppRunHistory
#396
feat: Introduce MaximumFollowNextPageLink option for usage with Get-AzLogicAppRunHistory
#396
Conversation
✅ Deploy Preview for arcus-scripting canceled.
|
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.
Again great stuff, descriptive tests with clear inputs. Super. 🎉
We can indeed, like you said, create a fix-release for this. |
Co-authored-by: Stijn Moreels <[email protected]>
Get-AzLogicAppRunHistory
…and `Resubmit-FailedAzLogicAppRuns`
…thub.com/pim-simons/arcus.scripting into feature/logic-app-follownextpagelink-max
src/Arcus.Scripting.LogicApps/Scripts/Cancel-AzLogicAppRuns.ps1
Outdated
Show resolved
Hide resolved
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.
Great!
Introduce MaximumFollowNextPageLink option in the config file for the Disable-AzLogicAppsFromConfig script.
This prevents retrieving all Logic App runs which can take a very long time.
If the value is not supplied in the config file a default of 10 is used, 30 results are returned per call, so the default will return a max of 300 Logic App runs, which should be enough for most use-cases I think.
As mentioned here #395