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

Support higher level constructs for AppSync #4277

Closed
1 of 2 tasks
hoegertn opened this issue Sep 28, 2019 · 6 comments
Closed
1 of 2 tasks

Support higher level constructs for AppSync #4277

hoegertn opened this issue Sep 28, 2019 · 6 comments
Assignees
Labels
@aws-cdk/aws-appsync Related to AWS AppSync feature-request A feature should be added or improved. needs-discussion This issue/PR requires more discussion with community. package/cfn Related to the CFN layer (L1)

Comments

@hoegertn
Copy link
Contributor

I propose the implementation of higher-level constructs for AppSync. Is this something that other people would find helpful?

Use Case

It would make implementing applications using AppSync easier than using the low-level objects.

Proposed Solution

This is subject to discussion. I am developing an AppSync application right now and would create higher-level constructs on the way if this is of interest.

Other

  • 👋 I may be able to implement this feature request
  • ⚠️ This feature might incur a breaking change

This is a 🚀 Feature Request

@hoegertn hoegertn added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Sep 28, 2019
@hoegertn
Copy link
Contributor Author

@eladb @rix0rrr What do you think?

@SomayaB SomayaB added the @aws-cdk/aws-appsync Related to AWS AppSync label Sep 30, 2019
@NGL321 NGL321 added needs-discussion This issue/PR requires more discussion with community. package/cfn Related to the CFN layer (L1) and removed needs-triage This issue or PR still needs to be triaged. labels Oct 4, 2019
@hoegertn
Copy link
Contributor Author

hoegertn commented Oct 7, 2019

@shivlaks Any comment on this proposal? IF this is of interest I would push a first draft implementation this week. It is a preliminary result of my current AppSync application.

@ranguard
Copy link
Contributor

ranguard commented Oct 9, 2019

@hoegertn I was just going to share https://github.com/nestorstay/cdk-constructs/tree/master/constructs/appsync which is a very basic (but working) set of wrappers around Cfn for AppSync, currently with only DynDB datasource
support.

Would be great to get a @aws-cdk/cdk-appsync package, so my advise could be... don't ask for permission to submit... just submit and see what happens :)

@hoegertn
Copy link
Contributor Author

hoegertn commented Oct 9, 2019

@ranguard Ok, I will submit my work today as a WIP PR and then we can discuss it.

@hoegertn
Copy link
Contributor Author

see #4438

@hoegertn
Copy link
Contributor Author

hoegertn commented Dec 6, 2019

This is merged into master

@hoegertn hoegertn closed this as completed Dec 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-appsync Related to AWS AppSync feature-request A feature should be added or improved. needs-discussion This issue/PR requires more discussion with community. package/cfn Related to the CFN layer (L1)
Projects
None yet
Development

No branches or pull requests

5 participants