brianhorakh / vita

vita source
0 stars 0 forks source link

-Comments on media should display actual username of person writing comment instead of the actual word "username" #12

Open inder243 opened 8 years ago

inder243 commented 8 years ago

Not getting username in Eventlist api

brianhorakh commented 8 years ago

The UID of the user is returned in the EventList, please use UserProfilesInfo with the UID to lookup both username + user thumbnail.

inder243 commented 8 years ago

We completed understand what you want us to do, but for the same we need to hit two service simultaneously to get username and thumbnail for each media. The service will be hit everytime an event is displayed. This is slow down the application when huge set of data comes in, eventually reducing the application process time and there by will drain battery. (Already we are facing battery issue at our end due to huge set of data to be handled.)

brianhorakh commented 8 years ago

Alright I'm going to re-assign this to myself. I'll include a summary of UserProfile info in the EventList corresponding to the UID's with each request. This may add additional processing time.

brianhorakh commented 8 years ago

This has been completed. Please look for the 'Users' node in the EventList response.

inder243 commented 8 years ago

Please check getting this in response: {"success":false,"error_msg":"internal server error Can't locate object method \"fetchrow_hashref\" via package \"DBI::db\" at /httpd/vita/perllib/VITA.pm line 1302.\n","*debug-sql":"select \ from Events E where 1=1 and E.uid='100' or E.evid in ('546571F6-FC86-11E5-9712-D99FB462889C','D912F9C0-0166-11E6-A6E7-D99FB462889C','DD81D64A-02DB-11E6-A33D-D99FB462889C'

Please correct the same, so that we can check the same.

brianhorakh commented 8 years ago

sorry about that, please try now.

inder243 commented 8 years ago

Thank you for adding users in the API, but its doesn't solve our problem, we have to check with each media which user has made comment.(This will again increasing the processing time, we will need to compare each UID corresponding to each mediaID.) It will be really beneficial for us if you can provide Username and Profile with corresponding UID in each media.

bumps = 0; created = "2016-06-05 22:43:08"; evid = "274208AC-2BAB-11E6-9ECF-88D178BD133F"; "media_url" = "http://www.vitatheapp.com/files/2016-06-06/05/274208AC-2BAB-11E6-9ECF-88D178BD133F.503.mp4"; mediaid = 503; "thumb_url" = "http://www.vitatheapp.com/files/2016-06-06/05/274208AC-2BAB-11E6-9ECF-88D178BD133F-thumb.503.jpg"; uid = 100; url = "http://www.vitatheapp.com/files/2016-06-06/05/274208AC-2BAB-11E6-9ECF-88D178BD133F.503.mp4";

Here Above If under UID: you can add two parameters UserName and ProfilePic. That will subsequently decrease time and will help us fetching data easily.

Please do add username,UID and ProfilePic of users who have commented for given media too.

SeanGuerrero commented 8 years ago

Brian, please see this one as well.

brianhorakh commented 8 years ago

Please see the response in issue #18

Please check the 'Users' section of the EventList response, for every uid returned to you, there will be a matching row in 'Users'