Skip to content

Commit

Permalink
Merge pull request #425 from edx-solutions/release
Browse files Browse the repository at this point in the history
Merge Release back into master
Conflicts:
	cms/envs/aws.py
	requirements/edx/custom.txt
  • Loading branch information
Fred Smith authored and Kelketek committed Aug 5, 2015
1 parent cc438a3 commit 58aa413
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
8 changes: 4 additions & 4 deletions lms/djangoapps/social_engagement/engagement.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,11 +113,11 @@ def _compute_social_engagement_score(social_metrics):
'SOCIAL_METRIC_POINTS',
{
'num_threads': 10,
'num_comments': 2,
'num_replies': 1,
'num_upvotes': 5,
'num_comments': 15,
'num_replies': 15,
'num_upvotes': 25,
'num_thread_followers': 5,
'num_comments_generated': 1,
'num_comments_generated': 15,
}
)

Expand Down
6 changes: 3 additions & 3 deletions lms/djangoapps/social_engagement/tests/test_engagement.py
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ def test_update_user_engagement_score(self):

self.assertEqual(
leaderboard_position['score'],
24
85
)

self.assertEqual(
Expand Down Expand Up @@ -264,7 +264,7 @@ def test_multiple_users(self):

self.assertEqual(
leaderboard_position['score'],
24
85
)

# user should be in place #2
Expand All @@ -282,7 +282,7 @@ def test_multiple_users(self):

self.assertEqual(
leaderboard_position['score'],
48
170
)

# user2 should be in place #1
Expand Down

0 comments on commit 58aa413

Please sign in to comment.