Skip to content

Fix Web README with local testing instructions - #541

Merged
seshubaws merged 2 commits into
masterfrom
seshubaws-patch-1
Jan 8, 2026
Merged

seshubaws merged 2 commits into
masterfrom
seshubaws-patch-1

Conversation

@seshubaws

@seshubaws seshubaws commented Jan 7, 2026 •

Copy link
Copy Markdown
Contributor

Updated README.md to include instructions for configuring env.json with DynamoDB table physical ID and running the local API. Removed references to endpoint override as that is not used and not necessary in the node Web template, as well as updated other steps for clarity. Clarified some steps in the full stack template readme as well. I manually ran through all the local invoke steps in both templates to verify they worked.

Issue #, if available:
#538
Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updated README.md to include instructions for configuring env.json with DynamoDB table physical ID and running the local API.
@seshubaws
seshubaws requested a review from a team as a code owner January 7, 2026 23:03
```
6. For testing - you can put an item into dynamodb local
4. Create the DynamoDB table (sample command below):
```

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Any reason why we're creating DDB table after starting sam local api

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

When I was testing, I found it didn't matter what order we did these ops in

@seshubaws
seshubaws added this pull request to the merge queue Jan 8, 2026
Merged via the queue into master with commit bab8b11 Jan 8, 2026
50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants