Conduit_NodeJS

Conduit_NodeJS copied to clipboard

It's been over a year since the last time this repo is updated. I wonder if this repo is still maintained.

kevinchristianto

Hi! I'm sniffing your code for learning purposes and found 2 logs hanging around. Are they really supposed to be there? lol controllers -> articles.js lines 92 and 91.

ivanzigoni

Business logic in controllers

1

I'm trying to figure out the best nodejs architecture for building APIs, and I see here that in the provided example, the controllers handle the business logic instead of using...

garyscetbon

Hello, By launching the Realworld Postman collection to test this API (https://conduit-api-realworld.herokuapp.com), there are 147 assertion errors. This API doesn't match the Realwolrd API specs requirements. Please fix it so...

geromegrignon

The sql query string for feed api throw error 'relation "followers" does not exist', because the 'followers' should be 'Followers'. This PR also add double quotes to words containing uppercase...

Requiemu

According to API specification, updating user information should use 'put' http method rather than 'patch' method.

Requiemu