Repository: oppia/oppia

Issue:
Remove the "Recently Published" section from the Library page
As a learner, I want to find explorations that are relevant and of high quality so I can learn about something I'm interested in. The "Recently Published" category on the Library page is generally filled with explorations that are neither relevant to the average user nor of particularly high quality. 

We should remove this section from this page.
Remove the "Recently Published" section from the Library page
As a learner, I want to find explorations that are relevant and of high quality so I can learn about something I'm interested in. The "Recently Published" category on the Library page is generally filled with explorations that are neither relevant to the average user nor of particularly high quality. 

We should remove this section from this page.


Discussion:
I'd like to take this up.
Howdy @cosmicAsymmetry ! Feel free to assign yourself. I noticed that you have some work assigned to you for another issue. Might it be better to complete that first or is that work on hold?
Yes, I do. I have submitted the PR for that and am waiting for the review to happen. Since I have some time on my hands(summer break :p) I'm trying to contribute as much as I can. Although I'll try not to work on more than two issues at once as it might get confusing.
@cosmicAsymmetry -- it is indeed a good idea to do at most two things at a time, I think, especially since you're starting out! However, I think this issue is a bit complex to start with, so I want to suggest taking a look at #383 first and getting some experience navigating the frontend code. Would that be OK? Once you have some experience with both backend and frontend in isolation, then let's try a full-stack project, but I think it makes sense to "build up" a bit first with more self-contained projects.
I'd like to work on this.
I'd like to take this up.
Howdy @cosmicAsymmetry ! Feel free to assign yourself. I noticed that you have some work assigned to you for another issue. Might it be better to complete that first or is that work on hold?
Yes, I do. I have submitted the PR for that and am waiting for the review to happen. Since I have some time on my hands(summer break :p) I'm trying to contribute as much as I can. Although I'll try not to work on more than two issues at once as it might get confusing.
@cosmicAsymmetry -- it is indeed a good idea to do at most two things at a time, I think, especially since you're starting out! However, I think this issue is a bit complex to start with, so I want to suggest taking a look at #383 first and getting some experience navigating the frontend code. Would that be OK? Once you have some experience with both backend and frontend in isolation, then let's try a full-stack project, but I think it makes sense to "build up" a bit first with more self-contained projects.
I'd like to work on this.

Resolving patch:
diff --git a/core/controllers/library.py b/core/controllers/library.py
--- a/core/controllers/library.py
+++ b/core/controllers/library.py
@@ -105,9 +105,6 @@ def get(self):
         # TODO(sll): Support index pages for other language codes.
         summary_dicts_by_category = summary_services.get_library_groups([
             feconf.DEFAULT_LANGUAGE_CODE])
-        recently_published_summary_dicts = (
-            summary_services.get_recently_published_exp_summary_dicts(
-                feconf.RECENTLY_PUBLISHED_QUERY_LIMIT_FOR_LIBRARY_PAGE))
         top_rated_activity_summary_dicts = (
             summary_services.get_top_rated_exploration_summary_dicts(
                 [feconf.DEFAULT_LANGUAGE_CODE],
@@ -121,15 +118,6 @@ def get(self):
             user_settings = user_services.get_user_settings(self.user_id)
             preferred_language_codes = user_settings.preferred_language_codes
 
-        if recently_published_summary_dicts:
-            summary_dicts_by_category.insert(0, {
-                'activity_summary_dicts': recently_published_summary_dicts,
-                'categories': [],
-                'header_i18n_id': feconf.LIBRARY_CATEGORY_RECENTLY_PUBLISHED,
-                'has_full_results_page': True,
-                'full_results_url': feconf.LIBRARY_RECENTLY_PUBLISHED_URL,
-                'protractor_id': 'recently-published',
-            })
         if top_rated_activity_summary_dicts:
             summary_dicts_by_category.insert(0, {
                 'activity_summary_dicts': top_rated_activity_summary_dicts,


Test patch:
diff --git a/core/tests/protractor/explorationFeedback.js b/core/tests/protractor/explorationFeedback.js
--- a/core/tests/protractor/explorationFeedback.js
+++ b/core/tests/protractor/explorationFeedback.js
@@ -34,7 +34,7 @@ var workflow = require('../protractor_utils/workflow.js');
 describe('ExplorationFeedback', function() {
   var EXPLORATION_TITLE = 'Sample Exploration';
   var EXPLORATION_OBJECTIVE = 'To explore something';
-  var EXPLORATION_CATEGORY = 'Random';
+  var EXPLORATION_CATEGORY = 'Algorithms';
   var EXPLORATION_LANGUAGE = 'English';
 
   beforeEach(function() {
diff --git a/core/tests/protractor/libraryPagesTour.js b/core/tests/protractor/libraryPagesTour.js
--- a/core/tests/protractor/libraryPagesTour.js
+++ b/core/tests/protractor/libraryPagesTour.js
@@ -26,7 +26,7 @@ var workflow = require('../protractor_utils/workflow.js');
 describe('Library pages tour', function() {
   var EXPLORATION_TITLE = 'Test Exploration';
   var EXPLORATION_OBJECTIVE = 'To learn testing';
-  var EXPLORATION_CATEGORY = 'Random';
+  var EXPLORATION_CATEGORY = 'Algorithms';
   var EXPLORATION_LANGUAGE = 'English';
   var EXPLORATION_RATING = 4;
   var SEARCH_TERM = 'python';
@@ -34,6 +34,10 @@ describe('Library pages tour', function() {
     browser.get(general.LIBRARY_URL_SUFFIX);
   };
 
+  var visitRecentlyPublishedPage = function() {
+    browser.get('library/recently_published');
+  }
+
   it('visits the library index page', function() {
     visitLibraryPage();
   });
@@ -59,8 +63,7 @@ describe('Library pages tour', function() {
   });
 
   it('visits the recent explorations page', function() {
-    visitLibraryPage();
-    element(by.css('.protractor-test-library-recently-published')).click();
+    visitRecentlyPublishedPage();
     expect(browser.getCurrentUrl()).toContain('library/recently_published');
   });
 
diff --git a/core/tests/protractor/suggestions.js b/core/tests/protractor/suggestions.js
--- a/core/tests/protractor/suggestions.js
+++ b/core/tests/protractor/suggestions.js
@@ -27,7 +27,7 @@ var workflow = require('../protractor_utils/workflow.js');
 
 describe('Suggestions on Explorations', function() {
   var EXPLORATION_TITLE = 'Sample Exploration';
-  var EXPLORATION_CATEGORY = 'Random';
+  var EXPLORATION_CATEGORY = 'Algorithms';
   var EXPLORATION_OBJECTIVE = 'To explore something new';
   var EXPLORATION_LANGUAGE = 'English';