-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release 2.4.2 #3357
Release 2.4.2 #3357
Conversation
…om reader to classifier_services and classifier_services_test
…der bubble in editor. (#3277) * Change color combination required for visibility of reminder bubble in editor * Added scoped css * Add changes in css to make class unique
* Create ParamChange object * Lint * Convert exploration param changes to front end representation * converting exploration param changes to backend * Lint * Remove console logs * remove function * decouple fields in directive * move angular copy into factory * updates
….py in delete method (#3275) * Fixes : removed role from get request and used self.is_admin and self.is_moderator instead * added test to check the logging after deletion of exploration * Requested changes made in editor_test.py * comments added * comment updated
Fix #3282: Fixes the validate() function for LDAStringClassifier
Fix #3199: Constants loading refactor
…ions for creator dashboard (#3196) * Fix#2160 Introduce final version of tests involving deleting explorations for creator dashboard * Fix#2160 Added required changes for tests involving deleting exploration from creator's dashboard * Fixed #2160 final review needed for changes as suggested * Fix#2160 Removed errors mentioned in the log * Fix#2160 finally all tests are passed locally please review it * Fix #2160 ready to merge please review it * Fixed #2160 Done minor changes as suggested * Fixed#2160 Resolved Linting Errors * Fixed #2160 ready to merge please review
…aining private explorations. (#3351)
Codecov Report
@@ Coverage Diff @@
## master #3357 +/- ##
==========================================
+ Coverage 44.82% 45.96% +1.14%
==========================================
Files 230 240 +10
Lines 18644 18398 -246
Branches 3053 2955 -98
==========================================
+ Hits 8357 8457 +100
+ Misses 10287 9941 -346
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. (Wow, big release!)
No description provided.