oppia · brianrodri · Jan 1, 2019 · Aug 11, 2018 · Aug 12, 2018 · Aug 12, 2018
@@ -166,6 +166,14 @@
     "I18N_GET_STARTED_PAGE_PARAGRAPH_9": "When learners go through your exploration, they can send you feedback to alert you to problems or to share ideas for making it better.",
     "I18N_GET_STARTED_PAGE_PARAGRAPH_9_HEADING": "Improve Your Exploration",
     "I18N_GET_STARTED_PAGE_TITLE": "Get Started",
+    "I18N_IMPROVEMENTS_TAB_FEEDBACK_DESCRIPTION": "A learner had something to say about your exploration.",
+    "I18N_IMPROVEMENTS_TAB_PLAYTHROUGH_ISSUES_CYCLIC_TRANSITIONS_DESCRIPTION": "A player has quit an exploration after repeatedly cycling through a sequence of cards.",
+    "I18N_IMPROVEMENTS_TAB_PLAYTHROUGH_ISSUES_EARLY_QUIT_DESCRIPTION": "A player has quit an exploration soon after starting.",
+    "I18N_IMPROVEMENTS_TAB_PLAYTHROUGH_ISSUES_MULTIPLE_INCORRECT_ANSWERS_DESCRIPTION": "A player has quit an exploration after entering many incorrect answers.",
+    "I18N_IMPROVEMENTS_TAB_SUMMARY_ARCHIVED_ISSUES": "Archived Issues",
+    "I18N_IMPROVEMENTS_TAB_SUMMARY_OPEN_ISSUES": "Open Issues",
+    "I18N_IMPROVEMENTS_TAB_SUMMARY_RESOLVED_ISSUES": "Resolved Issues",
+    "I18N_IMPROVEMENTS_TAB_UNRESOLVED_ANSWER_DESCRIPTION": "A commonly entered answer has no explicit feedback from you.",
     "I18N_INTERACTIONS_FRACTIONS_INPUT_PLACEHOLDER": "Enter a fraction in the form x/x, or a mixed number in the form x x/x.",
     "I18N_INTERACTIONS_FRACTIONS_INPUT_PLACEHOLDER_NO_INTEGER": "Enter a fraction in the form x/x.",
     "I18N_INTERACTIONS_GRAPH_ADD_EDGE": "Add Edge",

@@ -166,6 +166,14 @@
 	"I18N_GET_STARTED_PAGE_PARAGRAPH_9": "Text on the Get Started page that explains the process for creating an Oppia activity at a high level.\n{{Identical|Get started}}",
 	"I18N_GET_STARTED_PAGE_PARAGRAPH_9_HEADING": "Paragraph heading on the Get Started page that explains the process for creating an Oppia activity at a high level.\n{{Identical|Get started}}",
 	"I18N_GET_STARTED_PAGE_TITLE": "Title of the Get Started page that explains the process for creating an Oppia activity at a high level.\n{{Identical|Get started}}",
+	"I18N_IMPROVEMENTS_TAB_FEEDBACK_DESCRIPTION": "Text displayed to describe an issue card describing a feedback thread about the exploration.",
+	"I18N_IMPROVEMENTS_TAB_PLAYTHROUGH_ISSUES_CYCLIC_TRANSITIONS_DESCRIPTION": "Text displayed to describe an issue card describing a playthrough demonstrating a learner that had quit after being stuck moving through the same set of cards repeatedly.",
+	"I18N_IMPROVEMENTS_TAB_PLAYTHROUGH_ISSUES_EARLY_QUIT_DESCRIPTION": "Text displayed to describe an issue card describing a playthrough demonstrating a learner that quit immediately after starting an exploration.",
+	"I18N_IMPROVEMENTS_TAB_PLAYTHROUGH_ISSUES_MULTIPLE_INCORRECT_ANSWERS_DESCRIPTION": "Text displayed to describe an issue card describing playthrough demonstrating a learner that quit after entering incorrect answers repeatedly.",
+	"I18N_IMPROVEMENTS_TAB_SUMMARY_ARCHIVED_ISSUES": "Text displayed in the summary card to show the number of issues that have been archived.",
+	"I18N_IMPROVEMENTS_TAB_SUMMARY_OPEN_ISSUES": "Text displayed in the summary card to show the number of issues that are open and require attention.",
+	"I18N_IMPROVEMENTS_TAB_SUMMARY_RESOLVED_ISSUES": "Text displayed in the summary card to show the number of issues that have been given attention and deemed fixed by Oppia.",
+	"I18N_IMPROVEMENTS_TAB_UNRESOLVED_ANSWER_DESCRIPTION": "A commonly entered answer has no explicit feedback from you.",
 	"I18N_INTERACTIONS_FRACTIONS_INPUT_PLACEHOLDER": "Text displayed in the fractions interaction input as a placeholder.",
 	"I18N_INTERACTIONS_FRACTIONS_INPUT_PLACEHOLDER_NO_INTEGER": "Text displayed in the fractions interaction input as a placeholder when no integer part is allowed.",
 	"I18N_INTERACTIONS_GRAPH_ADD_EDGE": "Text displayed in a button in a graph interaction inside the graph editor - When the user clicks the button, he can add a new line (edge) to the graph.",

@@ -878,3 +878,17 @@ def get(self, exploration_id):
         self.render_json({
             'unresolved_answers': unresolved_answers_with_frequency
         })
+
+
+class ExplorationFeaturesHandler(EditorHandler):
+    """Returns features the given exploration is configured to support."""
+
+    GET_HANDLER_ERROR_RETURN_TYPE = feconf.HANDLER_TYPE_JSON
+
+    @acl_decorators.can_play_exploration
+    def get(self, unused_exploration_id):
+        """Handles GET requests for an exploration's features."""
+        self.render_json({
+            'is_improvements_tab_enabled':
+                config_domain.IS_IMPROVEMENTS_TAB_ENABLED.value,
+        })
@@ -274,3 +274,8 @@ def get_config_property_schemas(cls):
 RECORD_PLAYTHROUGH_PROBABILITY = ConfigProperty(
     'record_playthrough_probability', FLOAT_SCHEMA,
     'The probability of recording playthroughs', 0.2)
+
+IS_IMPROVEMENTS_TAB_ENABLED = ConfigProperty(
+    'is_improvements_tab_enabled', BOOL_SCHEMA,
+    'Exposes the Improvements Tab for creators in the exploration editor.',
+    False)
diff --git a/core/templates/dev/head/pages/collection_editor/CollectionEditorNavbarBreadcrumbDirective.js b/core/templates/dev/head/pages/collection_editor/CollectionEditorNavbarBreadcrumbDirective.js
@@ -41,14 +41,15 @@ oppia.directive('collectionEditorNavbarBreadcrumb', [
             preview: 'Preview',
             settings: 'Settings',
             stats: 'Statistics',
-            history: 'History'
+            improvements: 'Improvements',
+            history: 'History',
           };
 
           $scope.collection = CollectionEditorStateService.getCollection();
 
           $scope.getCurrentTabName = function() {
             return _TAB_NAMES_TO_HUMAN_READABLE_NAMES[
-              RouterService.getTabStatuses().active];
+              RouterService.getActiveTabName()];
           };
 
           $scope.editCollectionTitle = function() {

diff --git a/core/templates/dev/head/pages/collection_editor/CollectionEditorNavbarDirective.js b/core/templates/dev/head/pages/collection_editor/CollectionEditorNavbarDirective.js
@@ -49,7 +49,7 @@ oppia.directive('collectionEditorNavbar', [
           $scope.isSaveInProgress = (
             CollectionEditorStateService.isSavingCollection);
 
-          $scope.getTabStatuses = RouterService.getTabStatuses;
+          $scope.getActiveTabName = RouterService.getActiveTabName;
           $scope.selectMainTab = RouterService.navigateToMainTab;
           $scope.selectPreviewTab = RouterService.navigateToPreviewTab;
           $scope.selectSettingsTab = RouterService.navigateToSettingsTab;

diff --git a/core/templates/dev/head/pages/collection_editor/collection_editor.html b/core/templates/dev/head/pages/collection_editor/collection_editor.html
@@ -28,19 +28,19 @@
 
 {% block content %}
   <div ng-controller="CollectionEditor">
-    <div ng-if="getTabStatuses().active === 'main'">
+    <div ng-if="getActiveTabName() === 'main'">
       <collection-main-tab></collection-main-tab>
     </div>
 
-    <div ng-show="getTabStatuses().active === 'settings'">
+    <div ng-show="getActiveTabName() === 'settings'">
       <collection-settings-tab></collection-settings-tab>
     </div>
 
-    <div ng-show="getTabStatuses().active === 'history'">
+    <div ng-show="getActiveTabName() === 'history'">
       <collection-history-tab></collection-history-tab>
     </div>
 
-    <div ng-show="getTabStatuses().active === 'stats'">
+    <div ng-show="getActiveTabName() === 'stats'">
       <collection-statistics-tab></collection-statistics-tab>
     </div>
   </div>
@@ -134,6 +134,7 @@
   <script src="/templates/dev/head/pages/exploration_editor/ExplorationDataService.js"></script>
   <script src="/templates/dev/head/pages/exploration_editor/AngularNameService.js"></script>
   <script src="/templates/dev/head/pages/exploration_editor/ExplorationEditor.js"></script>
+  <script src="/templates/dev/head/pages/exploration_editor/ExplorationFeaturesService.js"></script>
   <script src="/templates/dev/head/pages/exploration_editor/RouterService.js"></script>
   <script src="/templates/dev/head/pages/exploration_editor/ChangesInHumanReadableFormService.js"></script>
 

diff --git a/core/templates/dev/head/pages/collection_editor/collection_editor_navbar_directive.html b/core/templates/dev/head/pages/collection_editor/collection_editor_navbar_directive.html
@@ -1,6 +1,6 @@
 <ul class="nav navbar-nav oppia-navbar-nav navbar-right ng-cloak" style="margin-left: 20px;">
   <!-- Editor -->
-  <li ng-class="{'active': getTabStatuses().active === 'main', 'uib-dropdown': getWarningsCount()}" ng-attr-uib-dropdown="<[getWarningsCount()]>">
+  <li ng-class="{'active': getActiveTabName() === 'main', 'uib-dropdown': getWarningsCount()}" ng-attr-uib-dropdown="<[getWarningsCount()]>">
     <a href="#" uib-tooltip="Editor" ng-click="selectMainTab()" class="protractor-test-main-tab">
       <i class="material-icons">&#xE254;</i>
     </a>
@@ -22,7 +22,7 @@
   </li>
 
   <!-- Settings -->
-  <li ng-class="{'active': getTabStatuses().active === 'settings'}">
+  <li ng-class="{'active': getActiveTabName() === 'settings'}">
     <a href="#" uib-tooltip="Settings" tooltip-placement="bottom" ng-click="selectSettingsTab()" class="protractor-test-settings-tab">
       <i class="material-icons">&#xE8B8;</i>
     </a>

diff --git a/core/templates/dev/head/pages/creator_dashboard/creator_dashboard.html b/core/templates/dev/head/pages/creator_dashboard/creator_dashboard.html
@@ -938,7 +938,7 @@ <h2>Suggestions to review</h2>
   <script src="/templates/dev/head/services/ComputeGraphService.js"></script>
   <script src="/templates/dev/head/services/EditabilityService.js"></script>
   <script src="/templates/dev/head/services/GenerateContentIdService.js"></script>
-  <script src="/templates/dev/head/services/IssuesService.js"></script>
+  <script src="/templates/dev/head/services/ImprovementsService.js"></script>
   <script src="/templates/dev/head/services/LocalStorageService.js"></script>
   <script src="/templates/dev/head/services/SiteAnalyticsService.js"></script>
 

diff --git a/core/templates/dev/head/pages/exploration_editor/EditorNavbarBreadcrumbDirective.js b/core/templates/dev/head/pages/exploration_editor/EditorNavbarBreadcrumbDirective.js
@@ -51,16 +51,17 @@ oppia.directive('editorNavbarBreadcrumb', [
             preview: 'Preview',
             settings: 'Settings',
             stats: 'Statistics',
+            improvements: 'Improvements',
             history: 'History',
-            feedback: 'Feedback'
+            feedback: 'Feedback',
           };
 
           $scope.getCurrentTabName = function() {
-            if (!RouterService.getTabStatuses()) {
+            if (!RouterService.getActiveTabName()) {
               return '';
             } else {
               return _TAB_NAMES_TO_HUMAN_READABLE_NAMES[
-                RouterService.getTabStatuses().active];
+                RouterService.getActiveTabName()];
             }
           };
         }

diff --git a/core/templates/dev/head/pages/exploration_editor/EditorNavigationDirective.js b/core/templates/dev/head/pages/exploration_editor/EditorNavigationDirective.js
@@ -24,21 +24,23 @@ oppia.directive('editorNavigation', [
       templateUrl: UrlInterpolationService.getDirectiveTemplateUrl(
         '/pages/exploration_editor/editor_navigation_directive.html'),
       controller: [
-        '$scope', '$rootScope', '$timeout', '$uibModal', 'ContextService',
-        'ExplorationRightsService', 'ExplorationWarningsService',
-        'RouterService', 'StateEditorTutorialFirstTimeService',
-        'SiteAnalyticsService', 'ThreadDataService', 'UserService',
-        'WindowDimensionsService',
+        '$rootScope', '$scope', '$timeout', '$uibModal', 'ContextService',
+        'ExplorationFeaturesService', 'ExplorationRightsService',
+        'ExplorationWarningsService', 'RouterService', 'SiteAnalyticsService',
+        'StateEditorTutorialFirstTimeService', 'ThreadDataService',
+        'UserService', 'WindowDimensionsService',
         function(
-            $scope, $rootScope, $timeout, $uibModal, ContextService,
-            ExplorationRightsService, ExplorationWarningsService,
-            RouterService, StateEditorTutorialFirstTimeService,
-            SiteAnalyticsService, ThreadDataService, UserService,
-            WindowDimensionsService) {
+            $rootScope, $scope, $timeout, $uibModal, ContextService,
+            ExplorationFeaturesService, ExplorationRightsService,
+            ExplorationWarningsService, RouterService, SiteAnalyticsService,
+            StateEditorTutorialFirstTimeService, ThreadDataService,
+            UserService, WindowDimensionsService) {
           $scope.popoverControlObject = {
             postTutorialHelpPopoverIsShown: false
           };
           $scope.isLargeScreen = (WindowDimensionsService.getWidth() >= 1024);
+          $scope.isImprovementsTabEnabled =
+            ExplorationFeaturesService.isImprovementsTabEnabled;
 
           $scope.$on('openPostTutorialHelpPopover', function() {
             if ($scope.isLargeScreen) {
@@ -105,12 +107,14 @@ oppia.directive('editorNavigation', [
             ExplorationWarningsService.hasCriticalWarnings);
 
           $scope.ExplorationRightsService = ExplorationRightsService;
-          $scope.getTabStatuses = RouterService.getTabStatuses;
+          $scope.getActiveTabName = RouterService.getActiveTabName;
           $scope.selectMainTab = RouterService.navigateToMainTab;
           $scope.selectTranslationTab = RouterService.navigateToTranslationTab;
           $scope.selectPreviewTab = RouterService.navigateToPreviewTab;
           $scope.selectSettingsTab = RouterService.navigateToSettingsTab;
           $scope.selectStatsTab = RouterService.navigateToStatsTab;
+          $scope.selectImprovementsTab =
+            RouterService.navigateToImprovementsTab;
           $scope.selectHistoryTab = RouterService.navigateToHistoryTab;
           $scope.selectFeedbackTab = RouterService.navigateToFeedbackTab;
           $scope.getOpenThreadsCount = ThreadDataService.getOpenThreadsCount;