Cale/xmodule assets fewer deletions - #173
Merged
Merged
Conversation
This was referenced Jun 15, 2013
Merged
Rake by default imports all .rake files in the rakelib dir, so we can use that rather than doing our own import loop.
`rake install_prereqs` requires a minimal level of ruby and rake already installed. If it doesn't exist, then print out a helpful message indicating next steps.
The preprocess task requires system and env arguments in order to correctly load up the django environment to preprocess sass files to inject themes. In order for that task to recieve the arguments, all tasks that depend on it also have to accept that same set of arguments. This will all go away once the next evolution of themes arrives, which will remove the preprocessing needed to inject theme names.
When running under watchmedo, coffee doesn't display any useful information when it recompiles a changed file, so we make watchmedo echo that information instead.
xmodule-assets creates coffeescript files in the output directories. On its next run, it used to delete the javascript files compiled from those coffee files. Now it doesn't which should make coffee have to do less work. Fixes LMS-451
chrisrossi
pushed a commit
to jazkarta/edx-platform
that referenced
this pull request
Mar 31, 2014
This cleans up remaining issues w/ https://github.com/MITx/mitx/pull/155
aboudreault
pushed a commit
to aboudreault/edx-platform
that referenced
this pull request
Aug 14, 2014
…-user-groups-by-data ziafazal/api-filter-user-groups-by-data: added xblock_id filter
ataki
pushed a commit
to ataki/edx-platform
that referenced
this pull request
Mar 26, 2015
…cs_tab_switch Switched metrics tab data source to api client
prabhanshu
pushed a commit
to prabhanshu/edx-platform
that referenced
this pull request
Oct 13, 2018
* Restricted application statistics3 (openedx#173) * update code coverage settings * fixing code cov target * initial grades api implementation * adding v1 urls to lms * Added CourseAccess exception * adding self * Added initial pagination functionality * Initial functionality for pagination of bulk grades * Initial functionality for pagination of bulk grades * Fixing formatting * Fixing formatting * deleted to get old formatting back hopefully * adding back views * Fixing pagination * Improvements to bulk grades api * Refactored and added organization filtering * Adding tests * Adding more tests * removing extra indent * Patching view read/create function * Restricted oauth changes * Test fixes * Build on travis * Patch tests with setting to auto expire access tokens * fixing tests and quality errors * Fixed grades test * Fixes quality error * Adding grades api improvements * Improvements to restricted oauth setup of grades api v1 * Fixing tests, bulk grades admin needs staff access not superuser * Removing BULK_GRADES_ADMIN setting * Fixing enrollment org_filter tests * 1. Added allowedusers filter to the restricted application model. 2. Get course grades if all user who are enrolled in a course. 3. New scope "statistics" to pull grades informationof all users enrolled in a course with elevated permission. 4. Added permission check to valdiate allowed users, ONLY if applicable. * adding the missing files * Fixing security issues and cleaning up unreached code paths * Fixing user function * Fixing testing strategy * Fixing grades test setup * Fixing test error (cherry picked from commit e600555) * Updating to trigger build * Updating to trigger build * Remove unused travis matrix, fix coverage running on test files, fix unit tests. * Actually removing unused test matrix * Test fixes, removing unused code. ++codecov * Fixing quality error
Sujeet1379
pushed a commit
to chandrudev/edx-platform
that referenced
this pull request
Nov 17, 2022
Danyal-Faheem
pushed a commit
to Danyal-Faheem/edx-platform
that referenced
this pull request
Jul 15, 2025
* RTL fixes * RTL Fixes Co-authored-by: RahatAli <rahatali@A006-00935.local>
macdiesel
pushed a commit
to macdiesel/edx-platform
that referenced
this pull request
Mar 30, 2026
Reverts openedx#173 Because, shifting to V2 of Forum started giving errors in production. It gave a rise to "Slow Queries" in production
macdiesel
pushed a commit
to macdiesel/edx-platform
that referenced
this pull request
Mar 30, 2026
…nedx#193) This PR brings the commits related to removal of waffle flag ENABLE_FORUM_V2 and dependency removal of cs_comments_service of Open edX to python forum repo. This PR is revert of openedx#186 Which was revert of openedx#173
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
1 participant
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.
Depends on #185