search

distributev / the-app

The App
MIT License
0 stars 2 forks source link

API /api/flows: #85

Open distributev opened 8 years ago

distributev commented 8 years ago
  1. Please correct following 2 APIs to point to /flows/ instead of /users/

(/users/ does not make sense in this API context)

PUT /api/users/:id => Update flow :id PATCH /api/users/:id => Update flow :id

  1. Why there are 2 different APIs which do the same thing Update flow :id (PUT and PATCH) - It should be only 1 API which is doing Update flow :id
luismanuel001 commented 8 years ago
  1. Fixed, it was just a typo.
  2. The PUT and PATCH are different conventions used for updating resources on a rest api, but I'll just remove PATCH since it isn't used that much.