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.

Sign up for GitHub

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

Jump to bottom

Fixed not getting all paginated results from AWS clients calls #593

Merged
merged 2 commits into from Aug 14, 2023
Merged

Fixed not getting all paginated results from AWS clients calls #593

merged 2 commits into from Aug 14, 2023

Conversation

rddimon
Copy link
Collaborator

@rddimon rddimon commented Aug 14, 2023

Fixes #587

Description of Issue Fixed
The release of the #592

Changes proposed in this pull request:

  • Added getAWSPagedResults function for the getting paget results
  • Added retryStrategy for all AWS clients
  • Updated unittests

@rddimon rddimon self-assigned this Aug 14, 2023
@sonarcloud
Copy link

sonarcloud bot commented Aug 14, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@amplify-self-hosted-runners
Copy link

Check Name Conclusion Summary Output
SonarCloud Code Analysis success Quality Gate passed output
test (14.x) success test (14.x) output
test (16.x) success test (16.x) output
test (18.x) success test (18.x) output
SonarCloud success No new alerts in code changed by this pull request output

@rddimon rddimon merged commit 1dcb082 into main Aug 14, 2023
5 checks passed
@rddimon rddimon deleted the release-7_1_2 branch August 14, 2023 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aoskotsky-amplify aoskotsky-amplify aoskotsky-amplify approved these changes

Assignees

@rddimon rddimon

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Failed to find a stack (v7.0.4)
2 participants
@rddimon @aoskotsky-amplify