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

feat(amazonq): Using AB variation value as customization name when overridden #6035

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

spfink
Copy link
Contributor

@spfink spfink commented Nov 15, 2024

Problem

When a customization is overridden via AB, the name of the customization is not shown to users

Solution

Uses the variation field from the customizationArnOverride feature as the name for the customization. This allows the UX to show a customization name when the customizationArn is overriden


License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@spfink spfink requested review from a team as code owners November 15, 2024 22:20
@justinmk3
Copy link
Contributor

@spfink spfink changed the title Using AB variation value as customization name when overridden type(feat): using ab variation value as customization name when overridden Nov 15, 2024
@spfink spfink force-pushed the finks/override branch 2 times, most recently from 3a74734 to f419a47 Compare November 15, 2024 22:59
@spfink spfink changed the title type(feat): using ab variation value as customization name when overridden type(feat): Using AB variation value as customization name when overridden Nov 15, 2024
@spfink spfink changed the title type(feat): Using AB variation value as customization name when overridden feat(amazonq): Using AB variation value as customization name when overridden Nov 17, 2024
@spfink spfink requested a review from justinmk3 November 20, 2024 22:26
@jpinkney-aws jpinkney-aws merged commit f91092a into aws:master Nov 22, 2024
22 of 25 checks passed
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.

4 participants