graphjs / graphjs-server

Server-side components of GraphJS at https://github.com/phonetworks/graphjs
https://graphjs.com
MIT License
47 stars 25 forks source link

getFollowers(id) and getFollowing(id) returns their results ignoring the id #32

Closed ozanilbey closed 6 years ago

ozanilbey commented 6 years ago

Sample requests:

http://phonetworks.com:1338/getFollowers?id=4774cd048bdf5e9f67a2fc3d67e46e89&public_id=79982844-6a27-4b3b-b77f-419a79be0e10 http://phonetworks.com:1338/getFollowers?id=ANYTHING&public_id=79982844-6a27-4b3b-b77f-419a79be0e10

http://phonetworks.com:1338/getFollowing?id=4774cd048bdf5e9f67a2fc3d67e46e89&public_id=79982844-6a27-4b3b-b77f-419a79be0e10 http://phonetworks.com:1338/getFollowing?id=ANYTHING&public_id=79982844-6a27-4b3b-b77f-419a79be0e10

esokullu commented 6 years ago

Currently getFollowers and getFollowing are session based, thus show the session owner's followers/following only.

If you check out https://github.com/phonetworks/graphjs-server/blob/master/src/GraphJS/Controllers/MembersController.php there is no such parameter as "id"

Do you need me to implement that?

ozanilbey commented 6 years ago

I thought that I answered this but apparently I forgot to do that. I definitely need this. Otherwise, it's not possible to see followers/following of other profiles.