advanced-interactivity-in-amp icon indicating copy to clipboard operation
advanced-interactivity-in-amp copied to clipboard

Described Endpoint (route) in Section 5: Working with Remote Data not correct

Open kkruups opened this issue 8 years ago • 0 comments

Problem:
Although, the endpoint described may be author's shorthand for describing a route which retrieves a shirt given a given a , the fact that an actual route as written does not exist in the file makes it a bit confusing to the reader. It is best providing the actual route than the description provided

Details App.js does not contain an Endpoint (route) as described in following paragraph:

Excerpt from Section 5:

...... Express.js development server in app.js already has an endpoint /shirts/sizes?shirt=<sku> which, given a shirt SKU, returns the available sizes and price for each size. It sends the response with an artificial delay of one second to simulate network latency. ....

Endpoint Described: /shirts/sizes?shirt=<sku>

Actual Endpoints (routes) found in App.js:

Enpoint For Get Routes

`/shirts/sizesAndPrices?sku=xxxx

` For Post Routes

`/shirts/addToCart`

Request: Correct Endpoint to actual endpoint provided

kkruups avatar Jun 05 '17 01:06 kkruups