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

feat(servicecatalog): Create TagOptions Construct #18314

Merged
merged 19 commits into from
Jan 25, 2022
Merged

feat(servicecatalog): Create TagOptions Construct #18314

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
d0e30d3
feat(servicecatalog): Create standalone TagOptions Construct
Jan 7, 2022
c3d930c
addressing some comments
Jan 14, 2022
023e440
quick fix
Jan 14, 2022
d519888
put modified tests back
Jan 14, 2022
e6cc8cb
Merge branch 'master' into tagOptionsConstruct
arcrank Jan 14, 2022
fb125c1
removing linting lines
Jan 14, 2022
f67a303
Merging in changes from mainline
Jan 14, 2022
d1cfcd4
fixing lint in package.json
Jan 14, 2022
58e9391
make props struct
Jan 14, 2022
f33d001
addressing comments
Jan 19, 2022
8c0f465
removing unintentional change
Jan 19, 2022
e434806
another unintentional typo
Jan 19, 2022
35e831f
one last doc cleanup
Jan 20, 2022
03d733a
add validation
Jan 20, 2022
e5bfdef
Merge branch 'master' into tagOptionsConstruct
arcrank Jan 21, 2022
e934d89
simplify iterating over unique tag values
Jan 21, 2022
ca9ec79
maintain current behavior for association resource naming
Jan 24, 2022
84f9410
merging from master
Jan 24, 2022
5f1f4a6
Merge branch 'master' into tagOptionsConstruct
mergify[bot] Jan 25, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
one last doc cleanup
  • Loading branch information
Aidan Crank committed Jan 20, 2022
commit 35e831f1a36ff90a9b9c250b81f344b7b9908cb1
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-servicecatalog/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,7 @@ const product = new servicecatalog.CloudFormationProduct(this, 'MyFirstProduct',
productVersions: [
{
productVersionName: "v1",
cloudFormationTemplate: servicecatalog.CloudFormationTemplate.fromProductStack(new S3BucketProduct(this, 'S3BucketProduct'),
cloudFormationTemplate: servicecatalog.CloudFormationTemplate.fromProductStack(new S3BucketProduct(this, 'S3BucketProduct')),
},
],
});
Expand Down
4 changes: 2 additions & 2 deletions packages/@aws-cdk/aws-servicecatalog/lib/tag-options.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export interface TagOptionsProps {
/**
* The values that are allowed to be set for specific tags.
* The keys of the map represent the tag keys,
* and the values of the map are a list of allowed values of that particular tag key.
* and the values of the map are a list of allowed values for that particular tag key.
*/
readonly allowedValuesForTags: { [tagKey: string]: string[] };
}
Expand All @@ -32,7 +32,7 @@ export class TagOptions extends cdk.Resource {
*
* @internal
*/
public readonly _tagOptionsMap: { [tagName: string]: CfnTagOption };
public readonly _tagOptionsMap: { [tagOptionIdentifier: string]: CfnTagOption };

constructor(scope: Construct, id: string, props: TagOptionsProps) {
super(scope, id);
arcrank marked this conversation as resolved.
Show resolved Hide resolved
Expand Down
2 changes: 1 addition & 1 deletion packages/@aws-cdk/aws-servicecatalog/test/portfolio.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -335,7 +335,7 @@ describe('portfolio associations and product constraints', () => {
Template.fromStack(stack).resourceCountIs('AWS::ServiceCatalog::TagOptionAssociation', 3);
}),

test('adding identical tag options to portfolio is idempotent', () => {
skinny85 marked this conversation as resolved.
Show resolved Hide resolved
test('adding tag options to portfolio multiple times is idempotent', () => {
const tagOptions = new servicecatalog.TagOptions(stack, 'TagOptions', {
allowedValuesForTags: {
key1: ['value1', 'value2'],
Expand Down