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

Fix issue with loading QuickSight dashboards #353

Merged
merged 1 commit into from
Dec 31, 2023

Conversation

reyes256
Copy link
Contributor

Fix QuickSight Dashboard Loading Issue

Description

This pull request addresses the issue with loading QuickSight dashboards in the Former2 tool. The problem stems from incorrect values being passed to the describe-dashboard-permissions method, leading to errors when retrieving created dashboards.

Changes Made

  • Corrected method parameters in the describe-dashboard-permissions call.
  • Ensured that the --dashboard-id parameter is used instead of the undefined analysis id.

Documentation Reference

Screenshots

emptydashboards
analysis
quicksight
image
listdashboards

Copy link
Owner

@iann0036 iann0036 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for your contribution!

@iann0036 iann0036 merged commit 4907bb0 into iann0036:master Dec 31, 2023
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.

None yet

2 participants