Add experimental crowdsource hinter module - #314
Merged
Conversation
Contributor
Author
Conflicts: common/static/coffee/src/logger.coffee
Added moderation feature - you can now choose to hold all hints for moderator approval before showing.
… the crowdsource hinter module. (Old tests no longer cover all the code, now that moderation has been added.)
…ier to read. Fixed database non-initialization bug in crowdsource hinter module.
…nsitive. Still very hacky.
Fixed some bugs in the tests for crowdsourced hinter.
Hinter now displays vote count after voting. Began testing templates.
Chiseled a little at writing template tests.
Began enforcing one-vote-per-person. This can be disabled with debug="True" in the <crowdsource_hinter> tag. Started tests of the hint manager.
…ents - this lets us test the hint manager.
…anager. Expanded tests of hint_manager. Enabled the hint_manager by default in development environments.
Expanded test coverage a little.
…dded some HTML formatting to same.
Fixed various commenting things. Removed an unused function in the crowdsourced module coffeescript. Improved commenting in hint_manager. Fixed pep and pylint violations.
…ror message instead of failing. Fixed bug when hinter module displays a hint, then is asked to display nothing. (Used to not update in this case.)
Limit the page and chapter numbers to digits, to keep malicious URL components from being inserted onto the page. Conflicts: lms/urls.py
Contributor
|
This has been deployed to production and can be merged |
rocha
added a commit
that referenced
this pull request
Jul 3, 2013
Add experimental crowdsource hinter module
chrisrossi
pushed a commit
to jazkarta/edx-platform
that referenced
this pull request
Mar 31, 2014
Style for CMS
e-kolpakov
referenced
this pull request
in open-craft/openedx-platform
Nov 14, 2014
…itle-when-empty ziafazal/api-update-user-title-when-empty:accept empty user title
smarnach
referenced
this pull request
in open-craft/openedx-platform
Sep 3, 2015
hachiyanagi-ks
added a commit
to nttks/edx-platform
that referenced
this pull request
Oct 2, 2015
hachiyanagi-ks
added a commit
to nttks/edx-platform
that referenced
this pull request
Oct 2, 2015
…x-messages-dashboard Fix messages for dashboard openedx#314
hachiyanagi-ks
added a commit
to nttks/edx-platform
that referenced
this pull request
Oct 6, 2015
…x-messages-dashboard Fix messages for dashboard openedx#314 (cherry picked from commit 16852b7) Conflicts: conf/locale/ja_JP/LC_MESSAGES/django.po conf/locale/ja_JP/LC_MESSAGES/mako.po
rediris
pushed a commit
to gymnasium/edx-platform
that referenced
this pull request
Feb 25, 2021
…rn.master/video-player-accessibility Amir/appsembler.hawthorn.master/video player accessibility
Sujeet1379
pushed a commit
to chandrudev/edx-platform
that referenced
this pull request
Nov 17, 2022
Co-authored-by: Renovate Bot <bot@renovateapp.com>
kiram15
pushed a commit
that referenced
this pull request
Jul 17, 2026
…n process (#314) Co-authored-by: alenkadev <alenka-sonata@2u.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.