LBRYCommunity/spee.ch

Trending images 2 #78

Merged

bones7242 merged 3 commits from trending-images-2 into master

2017-07-13 08:50:46 +02:00

bones7242 commented

2017-07-13 08:50:30 +02:00

(Migrated from github.com)

associated the File and Request tables (Files have many Requests, and Requests belong to one File)
for 'trending images', created one sql query that handles joining and grouping the requests rather than sorting the data in js

- associated the File and Request tables (Files have many Requests, and Requests belong to one File) - for 'trending images', created one sql query that handles joining and grouping the requests rather than sorting the data in js

kauffj (Migrated from github.com) reviewed 2017-07-13 16:53:50 +02:00

kauffj (Migrated from github.com) left a comment

Possible error

Possible error

controllers/statsController.js

kauffj (Migrated from github.com) commented

2017-07-13 16:52:13 +02:00

you probably want WHERE FileId IS NOT NULL

you probably want `WHERE FileId IS NOT NULL`

bones7242 (Migrated from github.com) reviewed 2017-07-28 06:41:36 +02:00

controllers/statsController.js

bones7242 (Migrated from github.com) commented

2017-07-28 06:41:36 +02:00

fixed 5ea7d959b9

fixed https://github.com/lbryio/spee.ch/commit/5ea7d959b9d960ef10000885a9867df477a0daf0

Sign in to join this conversation.

No reviewers

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/spee.ch#78

No description provided.