Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: update serverless and code to fix dynamodb local connection issues #719

Open
wants to merge 2 commits into
base: v3
Choose a base branch
from

Conversation

RichardWright
Copy link

Implement fixes mentioned in other threads

https://github.com/serverless/examples/issues/689

More to be added

Copy link

@medikoo medikoo left a comment

Choose a reason for hiding this comment

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

Thank you @RichardWright ! Please see my comments

@@ -61,7 +63,7 @@ functions:
path: todos/{id}
method: get
cors: true

Copy link

Choose a reason for hiding this comment

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

Let's not introduce whitespace changes

Copy link
Author

Choose a reason for hiding this comment

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

will remove

@RichardWright
Copy link
Author

Now updated

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants