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

Jump to bottom

fix(appsync): add missing pipelineConfig option on Resolver instanciation #6924

Closed
wants to merge 1 commit into from
Closed

fix(appsync): add missing pipelineConfig option on Resolver instanciation #6924

wants to merge 1 commit into from

Conversation

tcodes0
Copy link

@tcodes0 tcodes0 commented Mar 21, 2020 •
edited by SomayaB
Loading

Tentatively fixes #6923

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@tcodes0 tcodes0 changed the title fix(6923): add missing pipelineConfig option on Resolver instanciation fix(appsync): add missing pipelineConfig option on Resolver instanciation Mar 21, 2020
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject6AEA49D1-qxepHUsryhcu
  • Commit ID: a226763
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@tcodes0
Copy link
Author

tcodes0 commented Mar 21, 2020 •
edited
Loading

still failing overall but different error from cloudformation:

The specified functions [fieldArrayFilterLambda,fieldArrayLambda] must exist before referencing them from a resolver. (Service: AWSAppSync; Status Code: 404; Error Code: NotFoundException; Request ID: fca31651-8e71-4ec2-8e5b-827a32f265a7)

MrArnoldPalmer
MrArnoldPalmer requested changes Mar 26, 2020
View reviewed changes
Copy link
Contributor

@MrArnoldPalmer MrArnoldPalmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need a test case to cover this.

@MrArnoldPalmer MrArnoldPalmer mentioned this pull request Apr 6, 2020
Closed
@MrArnoldPalmer MrArnoldPalmer added the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. label Jun 29, 2020
@github-actions
Copy link

This PR has not received a response in a while. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

@github-actions github-actions bot added closing-soon This issue will automatically close in 4 days unless further comments are made. closed-for-staleness This issue was automatically closed because it hadn't received any attention in a while. and removed closing-soon This issue will automatically close in 4 days unless further comments are made. labels Jun 30, 2020
@github-actions github-actions bot closed this Jul 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MrArnoldPalmer MrArnoldPalmer MrArnoldPalmer requested changes

Assignees

@MrArnoldPalmer MrArnoldPalmer

Labels
closed-for-staleness This issue was automatically closed because it hadn't received any attention in a while. response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Pipeline config not showing up on synth output
3 participants
@tcodes0 @aws-cdk-automation @MrArnoldPalmer