Removes voice artist models and UI

oppia · Nik-09 · Apr 19, 2024 · Apr 22, 2024 · Jun 8, 2024 · Jun 9, 2024
commit 2214f8b78abaa99916d5ce76bac5a0ddf2480022
@@ -112,114 +112,6 @@ def put(self) -> None:
         self.render_json(self.values)
 
 
-class PutVoiceArtistMetadataHandlerNormalizedPayloadDict(TypedDict):
-    """Dict representation of VoiceArtistMetadataHandler's normalized_payload
-    dictionary.
-    """
-
-    voice_artist_id: str
-    language_code: str
-    language_accent_code: str
-
-
-class VoiceArtistMetadataHandler(
-    base.BaseHandler[
-        PutVoiceArtistMetadataHandlerNormalizedPayloadDict,
-        Dict[str, str]
-    ]
-):
-    """Handler class to manage voice artist data for the voiceover admin page.
-    """
-
-    GET_HANDLER_ERROR_RETURN_TYPE = feconf.HANDLER_TYPE_JSON
-    URL_PATH_ARGS_SCHEMAS: Dict[str, str] = {}
-    HANDLER_ARGS_SCHEMAS = {
-        'GET': {},
-        'PUT': {
-            'voice_artist_id': {
-                'schema': {
-                    'type': 'basestring'
-                }
-            },
-            'language_code': {
-                'schema': {
-                    'type': 'basestring'
-                }
-            },
-            'language_accent_code': {
-                'schema': {
-                    'type': 'basestring'
-                }
-            }
-        }
-    }
-
-    @acl_decorators.can_access_voiceover_admin_page
-    def get(self) -> None:
-        """Retrieves voice artist data for the voiceover admin page."""
-        voice_artist_id_to_language_mapping = (
-            voiceover_services.get_all_voice_artist_language_accent_mapping())
-        voice_artist_id_to_voice_artist_name = (
-            voiceover_services.get_voice_artist_ids_to_voice_artist_names())
-
-        self.values.update({
-            'voice_artist_id_to_language_mapping':
-                voice_artist_id_to_language_mapping,
-            'voice_artist_id_to_voice_artist_name':
-                voice_artist_id_to_voice_artist_name
-        })
-        self.render_json(self.values)
-
-    @acl_decorators.can_access_voiceover_admin_page
-    def put(self) -> None:
-        """Updates voice artist data from the voiceover admin page."""
-        assert self.normalized_payload is not None
-        voice_artist_id = self.normalized_payload['voice_artist_id']
-        language_code = self.normalized_payload['language_code']
-        language_accent_code = self.normalized_payload['language_accent_code']
-
-        voiceover_services.update_voice_artist_language_mapping(
-            voice_artist_id, language_code, language_accent_code)
-        self.render_json(self.values)
-
-
-class GetSampleVoiceoversForGivenVoiceArtistHandler(
-    base.BaseHandler[Dict[str, str], Dict[str, str]]
-):
-    """Handler class to get sample contributed voiceovers of a voice artist in
-    a given language.
-    """
-
-    GET_HANDLER_ERROR_RETURN_TYPE = feconf.HANDLER_TYPE_JSON
-    URL_PATH_ARGS_SCHEMAS = {
-        'voice_artist_id': {
-            'schema': {
-                'type': 'basestring'
-            }
-        },
-        'language_code': {
-            'schema': {
-                'type': 'basestring'
-            }
-        }
-    }
-    HANDLER_ARGS_SCHEMAS: Dict[str, Dict[str, str]] = {'GET': {}}
-
-    @acl_decorators.can_access_voiceover_admin_page
-    def get(self, voice_artist_id: str, language_code: str) -> None:
-        exploration_id_to_filenames = (
-            voiceover_services.get_voiceover_filenames(
-                voice_artist_id=voice_artist_id,
-                language_code=language_code
-            )
-        )
-
-        self.values.update({
-            'exploration_id_to_filenames': exploration_id_to_filenames,
-        })
-        self.render_json(self.values)
-
-
 class EntityVoiceoversBulkHandler(
     base.BaseHandler[Dict[str, str], Dict[str, str]]
 ):

@@ -2039,9 +2039,6 @@ def compute_models_to_put_when_saving_new_exp_version(
         )
     )
 
-    voiceover_services.update_exploration_voice_artist_link_model(
-        committer_id, change_list, old_exploration, updated_exploration)
-
     new_content_id_set = set(updated_exploration.get_translatable_content_ids())
     content_ids_corresponding_translations_to_remove = (
         old_content_id_set - new_content_id_set

@@ -477,7 +477,6 @@ def set_up_non_trivial(self) -> None:
         20) Creates new BlogPostModel and BlogPostRightsModel.
         21) Creates a TranslationContributionStatsModel.
         22) Creates new LearnerGroupModel and LearnerGroupsUserModel.
-        23) Creates a VoiceArtistMetadataModel.
         """
         # Setup for UserStatsModel.
         user_models.UserStatsModel(
@@ -973,11 +972,6 @@ def set_up_non_trivial(self) -> None:
             'en': 'en-US',
             'hi': 'hi-IN'
         }
-        # Setup for VoiceArtistMetadataModel.
-        voiceover_models.VoiceArtistMetadataModel.create_model(
-            voice_artist_id=self.USER_ID_1,
-            language_code_to_accent=language_code_to_accent
-        )
 
         # Set-up for AppFeedbackReportModel scrubbed by user.
         report_id = '%s.%s.%s' % (