Skip to content

feat: Added pspName parameter to quicksight dashboard url#539

Merged
svariant merged 2 commits intonextfrom
quicksight-dashboard-pspname-param
Feb 28, 2025
Merged

feat: Added pspName parameter to quicksight dashboard url#539
svariant merged 2 commits intonextfrom
quicksight-dashboard-pspname-param

Conversation

@svariant
Copy link
Collaborator

List of Changes

  • Added pspName parameter to quicksight dashboard url
  • Updated unit test

Motivation and Context

The pspName is needed in the quicksight dashboard

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as
    expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

Copy link
Collaborator

@gioelemella gioelemella left a comment

Choose a reason for hiding this comment

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

lgtm

@svariant svariant merged commit cf70bbf into next Feb 28, 2025
12 of 13 checks passed
@svariant svariant deleted the quicksight-dashboard-pspname-param branch February 28, 2025 10:06
@svariant svariant mentioned this pull request Mar 3, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants