Your Name
a36e3b1957
CourseEnrollment no longer needed in forum/views
2013-08-21 11:06:30 -04:00
Kevin Chugh
a2bbb65dcf
refactor to add access control to already_existing access control routines in access.py
2013-08-20 19:42:03 -04:00
David Ormsbee
013009ea24
Let staff have access to a forum even if they're not enrolled in the course.
2013-08-19 17:27:23 -04:00
David Ormsbee
c2d1791de4
Make it so that students who are not enrolled cannot see the forum
2013-08-19 17:09:47 -04:00
David Ormsbee
3ce87583ab
Shift enroll/unenroll logic to CourseEnrollment model, add is_active and mode.
...
Features coming down the pipe will want to be able to:
* Refer to enrollments before they are actually activated (approval step).
* See what courses a user used to be enrolled in for when they re-enroll in
the same course, or a different run of that course.
* Have different "modes" of enrolling in a course, representing things like
honor certificate enrollment, auditing (no certs), etc.
This change adds an is_active flag and mode (with default being "honor").
The commit is only as large as it is because many parts of the codebase were
manipulating enrollments by adding and removing CourseEnrollment objects
directly. It was necessary to create classmethods on CourseEnrollment to
encapsulate this functionality and then port everything over to using them.
The migration to add columns has been tested on a prod replica, and seems to be
fine for running on a live system with single digit millions of rows of
enrollments.
2013-08-14 13:23:06 -04:00
Kevin Chugh
e1e591f8bf
address code review comments
2013-07-30 01:55:20 -04:00
Your Name
6276997180
update checkbox with get settings
2013-07-29 13:26:31 -04:00
Kevin Chugh
d95693ebb7
hook up panel to new stats service
2013-07-25 03:23:04 -04:00
Sarina Canelake
e5f97b372b
pep8 fixes
2013-06-28 18:00:41 -04:00
Jay Zoldak
b1424a75b8
Add a test for 404 raised when requesting the profile page of a user that does not exist
2013-06-28 16:51:24 -04:00
Sarina Canelake
09501a62e2
Handle exception with a 404
2013-06-28 16:14:56 -04:00
Jay Zoldak
c98a77565f
Make the UrlResetMixin load the urlconf after resetting it, and fix the comment client test that was leaving ENABLE_DISCUSSION_SERVICE at True
2013-06-27 11:05:09 -04:00
Ned Batchelder
57909ce1aa
Fix all W0602, global used but no assignment done.
2013-06-21 17:33:23 -04:00
Ned Batchelder
df4b512b6f
Change wildcard imports into specific imports
2013-06-21 17:33:06 -04:00
Ned Batchelder
e045860cb6
Pylint complains if you use string, even if you use it for what its still meant to be used for.
2013-06-21 16:35:47 -04:00
Ned Batchelder
61b53713d2
Remove unused imports from lms, as detected by pylint.
2013-06-19 16:56:34 -04:00
Jason Bau
28ae2b6346
change form of making datetime.max aware (in UTC)
2013-06-17 16:11:35 -07:00
Jason Bau
da2fda1045
Fix for broken forums due to forum entry with no start_date
...
modification of @jrbl's version to handle a dict with value None
2013-06-15 01:30:42 -07:00
Calen Pennington
5b8163dc9c
Merge pull request #92 from edx/fix/cale/url-tests
...
Fix tests that vary urls.py
2013-06-12 10:47:13 -07:00
Don Mitchell
168cb306d3
Move tz_aware into connection config rather than settings.
...
Make django use_tz
2013-06-10 17:16:35 -04:00
Don Mitchell
d7194e6bec
struct_time to datetime conversion.
2013-06-10 17:16:34 -04:00
Calen Pennington
6c24694a7c
Fix tests that vary urls.py
...
Create a mixin class that can be used for tests that customize urls.py
to force django to reload it, so that they don't break other tests.
2013-06-10 10:56:50 -04:00
e0d
83a490c79d
fixing bug noticed in production
2013-06-06 16:04:42 -04:00
Chris Dodge
9b88528b04
Merge branch 'master' of github.com:edx/edx-platform into feature/cdodge/autoprovision-forums-master
2013-05-22 13:20:49 -04:00
Nate Hardison
cc7b2942ee
Handle Heroku's 503 maintenance mode response
...
The LMS comment client previously would try to parse the response
as JSON, choke, and return a 500 to the client. Now, the LMS client
displays a message indicating that the forums are down for
maintenance.
2013-05-21 10:03:51 -07:00
Chris Dodge
a44e816877
remove_open_ended_panel_tab() -> remove_extra_paneltab(). Also, update new forums test factors to use refactored Models.py
2013-05-20 10:33:52 -04:00
Chris Dodge
fbbae44988
Merge branch 'master' of github.com:edx/edx-platform into feature/cdodge/autoprovision-forums-master
...
Conflicts:
cms/djangoapps/contentstore/views/course.py
lms/djangoapps/django_comment_client/base/views.py
lms/djangoapps/django_comment_client/management/commands/show_permissions.py
lms/djangoapps/django_comment_client/models.py
lms/djangoapps/django_comment_client/tests/test_utils.py
lms/envs/common.py
2013-05-20 10:22:36 -04:00
Your Name
867d69ed12
update views.py to add docstrings and change exception name to address pylint errors
2013-05-16 18:13:59 -04:00
Jay Zoldak
71626f4ff6
Pep8 fixes of django_comment_client code
2013-05-16 15:57:34 -04:00
Your Name
a9db41d8d0
flag and unflag tests for comments and threads
2013-05-16 10:59:23 -04:00
Your Name
c7e0b576d0
fix missing paren
2013-05-16 10:59:23 -04:00
Kevin Chugh
03725de34a
flag abuse failing
2013-05-16 10:59:23 -04:00
Kevin Chugh
fa269d3c96
add flag thread django test
2013-05-16 10:59:23 -04:00
Jay Zoldak
e1481d93a8
Work on view test for comment client code.
2013-05-16 10:56:58 -04:00
Jay Zoldak
7e63611478
Start adding view tests for discussion client. Note that these will not work unless you change your test.py to enable discussion forums.
2013-05-16 10:56:57 -04:00
Jay Zoldak
ed5ad46192
Revert "Revert "fix merge conflict""
...
This reverts commit 2df3fe9344
2013-05-16 10:29:33 -04:00
Kevin Chugh
c410da91de
fix coffeescript error
2013-05-16 10:29:32 -04:00
Chris Dodge
5619306c6b
add comment that the models.py file has been moved
2013-05-10 13:43:15 -04:00
Chris Dodge
d39d44ba49
keep migrations in djangoapp_comment_client
2013-05-10 12:58:00 -04:00
Chris Dodge
11244107fd
add an empty models.py file to django_comment_client
2013-05-09 16:16:11 -04:00
Chris Dodge
40e3410121
fix broken unit tests
2013-05-06 16:51:36 -04:00
Chris Dodge
c20dd7d903
forgot to remove enough stuff from the django admin path
2013-05-06 16:40:25 -04:00
Chris Dodge
a92910db20
update lms to reflect where comment models.py is now in common
2013-05-06 16:23:32 -04:00
Chris Dodge
57eca325d2
move some files from lms/djangoapps/django_comment_client to common/djangoapps/django_comment_common
2013-05-06 15:59:35 -04:00
David Baumgold
b344976fe7
Making linters happier
2013-05-03 15:08:28 -04:00
Diana Huang
2df3fe9344
Revert "fix merge conflict"
...
This reverts commit acfd0d9fd6e4194c99d8
2013-04-26 12:40:35 -04:00
Your Name
068e02efd5
make unflag all permissions match javascript
2013-04-22 17:51:48 -04:00
Your Name
3f1d97b065
Merge branch 'master' into feature/kevin/flagging
2013-04-22 17:22:33 -04:00
Your Name
5e73e8f286
Merge branch 'master' of github.com:MITx/mitx
2013-04-22 17:22:24 -04:00
Your Name
30104979e7
fix downvoting url
2013-04-22 16:25:16 -04:00
