Skip to content

Add #exp_points to CourseUser model#175

Closed
kxmbrian wants to merge 9 commits intoCoursemology:masterfrom
coursemology-collab:add-course-user-exp
Closed

Add #exp_points to CourseUser model#175
kxmbrian wants to merge 9 commits intoCoursemology:masterfrom
coursemology-collab:add-course-user-exp

Conversation

@kxmbrian
Copy link
Contributor

Merge #173 first.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# for instance methods and . for class methods
http://betterspecs.org/#describe

@kxmbrian kxmbrian changed the title Add .exp_points to CourseUser model Add #exp_points to CourseUser model Apr 16, 2015
@kxmbrian kxmbrian force-pushed the add-course-user-exp branch from bfed13c to c1f5881 Compare April 16, 2015 05:04
New exp-awarding item (EA) cannot be saved unless ExpRecord is valid.
ExpRecord will not be valid until EA is saved.
@kxmbrian kxmbrian force-pushed the add-course-user-exp branch from c1f5881 to dcc8265 Compare April 16, 2015 13:25
@kxmbrian kxmbrian force-pushed the add-course-user-exp branch from dcc8265 to 0b6eefe Compare April 23, 2015 06:03
@kxmbrian
Copy link
Contributor Author

kxmbrian commented May 7, 2015

Replaced by #186, which is PR-ed from a name-spaced branch instead. Also, a cherry-picked commit was added.

@kxmbrian kxmbrian closed this May 7, 2015
@kxmbrian kxmbrian deleted the add-course-user-exp branch May 7, 2015 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants