biffgaut

Results 177 comments of biffgaut

New Pattern: aws-wafv2-appsync

Ah - I see your PR. I'll take a look - the nature of wafwebacl constructs requiring an already instantiated AppSync CDK construct reduces the risk of an eventual AppSync->(some...

New Pattern: aws-wafv2-appsync

As our WAF interface is all L1 constructs, that makes the inclusion of L1 AppSync constructs less concerning. If someone wants to use the Experimental AppSync construct, it should be...

New Pattern: aws-wafv2-appsync

(oops - wrong button...)

Pattern: aws-apigateway-sqs registers methods on root and a /message resource inconsistently

Thanks for the heads up - we'll take a look

Pattern: aws-apigateway-sqs registers methods on root and a /message resource inconsistently

This construct was written this way as one of the original constructs released in 2020. It does look odd to us now. We're looking back to see if there was...

Allow Lambda Aliases To Be Used With Constructs

Thanks - that's an interesting idea. We'll take a look.

Replace | any with Partial<T> for type checking across all constructs

That might work - we'll check it out. Thanks!

Replace | any with Partial<T> for type checking across all constructs

If we do it, it will be pretty massive - as we use this pattern EVERYWHERE, so it will affect virtually every construct (consistent behavior across constructs is one of...

Replace | any with Partial<T> for type checking across all constructs

Our next sprint meeting is Tuesday (delayed a day because of the US holiday). We'll discuss it as a team then - my guess is that we will take you...

Replace | any with Partial<T> for type checking across all constructs

We use a package called [JSII](https://github.com/aws/jsii) to create the interfaces on top of Constructs that allow them to be used in Java, Python, and (eventually) .NET. We copied this pattern...