sst.dev icon indicating copy to clipboard operation
sst.dev copied to clipboard

delete 'Remove Template Files' section

Open ClaytonFarr opened this issue 2 years ago • 3 comments

the initial API / hello world infrastructure is expected and built upon in subsequent tutorial steps, starting here - https://serverless-stack.com/chapters/review-our-app-architecture.html

ClaytonFarr avatar Sep 07 '21 17:09 ClaytonFarr

I might've missed this but in this PR (https://github.com/AnomalyInnovations/serverless-stack-com/pull/582) you are renaming MyStack.js. Are you also removing the src/lambda.js somewhere?

If not, then we can move that to the bottom part of the chapter in that PR.

jayair avatar Sep 07 '21 19:09 jayair

Ah, you're right. I was originally going to suggest updating the 'Add the Function' section (https://serverless-stack.com/chapters/add-an-api-to-create-a-note.html#add-the-function) to rename src/lambda.js to the new src/create.js file needed and update the file's contents, but had forgotten.

If you think it makes sense to direct the user to create a new file and delete the boilerplate lambda instead, it may be clearer to have that delete step occur the same chapter where we repurpose and rename MyStack.js.

ClaytonFarr avatar Sep 07 '21 21:09 ClaytonFarr

Yeah I think it makes more sense to not rename the boilerplate stack or lambda function. Just remove it. I'd suggest moving the section from this PR to the other one https://github.com/AnomalyInnovations/serverless-stack-com/pull/582. After we create the new API we can remove the old files?

jayair avatar Sep 10 '21 02:09 jayair