Added paginated curl to get all deployments for cloudflare pages #355
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deploying algokit-lora with
|
| Latest commit: |
a8554ae
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8c628322.algokit-lora.pages.dev |
| Branch Preview URL: | https://paginated-rm-preview-deploym.algokit-lora.pages.dev |
lempira
force-pushed
the
paginated-rm-preview-deployments
branch
from
6f9df40 to
d339a4b
Compare
neilcampbell
approved these changes
Nov 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I realized that the cloudflare workflow to delete the apps wasn't fully working. The endpoint is paginated and only allows 25 results per page. There is no param that allows for more results, just the "PAGE" pagination params. There is also no way to filter the results thru params.
I updated the worker to get all the results by looping through all the paginated results and then filtering the ones relevant to the branch.
I tested this on the current branch and it's working