Skip to content

Commit d8efc2a

Browse files
authored
Merge pull request #302 from nursoltan-s/recommender-final-fix-api
Recommender final fix api
2 parents a677ae6 + f8e98c2 commit d8efc2a

File tree

3 files changed

+34
-70
lines changed

3 files changed

+34
-70
lines changed

package-lock.json

Lines changed: 25 additions & 62 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
"lint:js": "./node_modules/.bin/eslint --ext .js,.jsx .",
3232
"test": "npm run lint && npm run jest"
3333
},
34-
"version": "1000.27.5",
34+
"version": "1.1.6",
3535
"dependencies": {
3636
"auth0-js": "^6.8.4",
3737
"config": "^3.2.0",

src/services/challenges.js

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -536,25 +536,26 @@ class ChallengesService {
536536
* @return {Promise} Resolves to the api response.
537537
*/
538538
async getRecommendedChallenges(filter, handle) {
539-
const query = getFilterUrl(
540-
filter.backendFilter,
541-
{ ...filter.frontFilter, per_page: filter.frontFilter.perPage },
542-
);
539+
filter.frontFilter.per_page = filter.frontFilter.perPage;
540+
delete filter.frontFilter.perPage;
541+
542+
const query = getFilterUrl(filter.backendFilter, filter.frontFilter);
543543

544544
let res = {};
545+
let totalCount = 0;
545546
if (_.some(filter.frontFilter.tracks, val => val)
546547
&& !_.isEqual(filter.frontFilter.types, [])) {
547548
const url = `/recommender-api/${handle}?${query}`;
548549
res = await this.private.apiV5.get(url).then(checkErrorV5);
550+
totalCount = res.headers.get('x-total') || 0;
549551
}
550-
const challenges = res.result.filter(ch => ch.jaccard_index > 0);
551552

552-
const totalCount = challenges.length;
553+
const challenges = res.result ? res.result.filter(ch => ch.jaccard_index > 0) : [];
553554
return {
554555
challenges,
555556
totalCount,
556557
meta: {
557-
allChallengesCount: challenges.length,
558+
allChallengesCount: totalCount,
558559
allRecommendedChallengesCount: 0,
559560
myChallengesCount: 0,
560561
ongoingChallengesCount: 0,

0 commit comments

Comments
 (0)