Skip to content

Commit

Permalink
chore: patch quicksight dataset property (#1256)
Browse files Browse the repository at this point in the history
RefreshConfiguration is marked as optional incorrectly by QuickSight
team. Updating it back by a temporary patch until their team fixed it.
  • Loading branch information
GavinZZ authored Aug 23, 2024
1 parent 9de7a85 commit bfa0c5e
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import './dms';
import './elasticsearch';
import './iot1click';
import './opensearch';
import './quicksight';
import './rds';
import './resiliencehub';
import './s3';
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
import { patching } from '@aws-cdk/service-spec-importers';
import { forResource, registerServicePatches, replaceDefinition } from './core';

registerServicePatches(
forResource('AWS::QuickSight::DataSet', (lens) => {
const reason = patching.Reason.sourceIssue(
'RefreshConfiguration property is marked as optional unintentionally.',
);
replaceDefinition(
'DataSetRefreshProperties',
{
type: "object",
description: "<p>The refresh properties of a dataset.</p>",
properties: {
RefreshConfiguration: {
"$ref": "#/definitions/RefreshConfiguration"
}
},
required: [ "RefreshConfiguration" ],
additionalProperties: false
},
reason,
)(lens);
})
);

0 comments on commit bfa0c5e

Please sign in to comment.