Skip to content
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.

Sign up for GitHub

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

Remove voiceover from exp schema #20675

Draft
wants to merge 39 commits into
base: develop
Choose a base branch
from
Draft

Remove voiceover from exp schema #20675

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
f003882
Adds conversions function for exploration migration.
Nik-09 Apr 19, 2024
71a0266
Removes recorded voiceover fields from exploration states backend
Nik-09 Apr 22, 2024
d000747
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Jun 8, 2024
68b14ad
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Jun 9, 2024
a930682
Fixes merge conflict
Nik-09 Jul 8, 2024
17f95f4
Removed recorded voiceover from exploration
Nik-09 Jul 9, 2024
d9e5a91
Removes feature flag
Nik-09 Jul 9, 2024
2214f8b
Removes voice artist models and UI
Nik-09 Jul 10, 2024
7f21430
Removes audio translation service files and uses.
Nik-09 Jul 14, 2024
d62e948
Fixes merge conflict
Nik-09 Jul 15, 2024
e7a8ac4
Fixes backend tests
Nik-09 Jul 15, 2024
16eacbc
Fixes backend tests
Nik-09 Jul 15, 2024
9fb7eca
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Jul 16, 2024
49380f3
Fixes frontend tests
Nik-09 Jul 21, 2024
439c490
Fixes merge conflict and frontend test
Nik-09 Jul 29, 2024
79c333b
Fixes frontend tests
Nik-09 Jul 29, 2024
7de0993
Fixes frontend tests
Nik-09 Aug 1, 2024
a348168
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Aug 1, 2024
827700f
Fixes lint issues
Nik-09 Aug 1, 2024
789cb77
Fixes lint issues
Nik-09 Aug 3, 2024
7519f17
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Aug 3, 2024
77ce7d1
Fixes merge conflict
Nik-09 Aug 7, 2024
23661e3
Fixes typescript checks
Nik-09 Aug 8, 2024
ff67301
Fixes merge conflict
Nik-09 Aug 18, 2024
4acb589
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Sep 4, 2024
8072d16
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Sep 21, 2024
3ea710d
Fixes frontend tests
Nik-09 Sep 23, 2024
a06d486
Fixes merge conflicts
Nik-09 Sep 30, 2024
c8a2218
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Oct 7, 2024
7f34239
Fixes merge conflict
Nik-09 Oct 30, 2024
11942df
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Nov 2, 2024
e89e16c
Fixes frontend tests
Nik-09 Nov 7, 2024
a8ab730
Fixes merge conflict
Nik-09 Nov 11, 2024
fc155ba
Reverts unnecessary changes.
Nik-09 Nov 24, 2024
bd34115
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Nov 24, 2024
95d39fa
Merge branch 'develop' of https://github.com/oppia/oppia into remove_…
Nik-09 Dec 1, 2024
c49c609
Fixes backend and lint checks
Nik-09 Dec 1, 2024
a87e250
Fixes merge conflict
Nik-09 Dec 10, 2024
446684c
Merge branch 'develop' into remove_voiceover_from_exp_schema
seanlip Dec 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removes recorded voiceover fields from exploration states backend
  • Loading branch information
@Nik-09
Nik-09 committed Apr 22, 2024
commit 71a02669b154c2c8972f8d5b6e8937203f7ec087
1 change: 0 additions & 1 deletion core/domain/change_domain.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@
state_domain.SolutionDict,
state_domain.StateDict,
state_domain.OutcomeDict,
state_domain.RecordedVoiceoversDict,
feconf.TranslatedContentDict,
question_domain.QuestionSuggestionChangeDict
]
Expand Down
46 changes: 3 additions & 43 deletions core/domain/exp_domain.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@
STATE_PROPERTY_CONTENT: Final = 'content'
STATE_PROPERTY_SOLICIT_ANSWER_DETAILS: Final = 'solicit_answer_details'
STATE_PROPERTY_CARD_IS_CHECKPOINT: Final = 'card_is_checkpoint'
STATE_PROPERTY_RECORDED_VOICEOVERS: Final = 'recorded_voiceovers'
DEPRECATED_STATE_PROPERTY_WRITTEN_TRANSLATIONS: Final = 'written_translations'
STATE_PROPERTY_INTERACTION_ID: Final = 'widget_id'
DEPRECATED_STATE_PROPERTY_NEXT_CONTENT_ID_INDEX: Final = 'next_content_id_index'
Expand Down Expand Up @@ -299,7 +298,6 @@ class ExplorationChange(change_domain.BaseChange):
STATE_PROPERTY_CONTENT,
STATE_PROPERTY_SOLICIT_ANSWER_DETAILS,
STATE_PROPERTY_CARD_IS_CHECKPOINT,
STATE_PROPERTY_RECORDED_VOICEOVERS,
STATE_PROPERTY_INTERACTION_ID,
STATE_PROPERTY_LINKED_SKILL_ID,
STATE_PROPERTY_INTERACTION_CUST_ARGS,
Expand Down Expand Up @@ -570,18 +568,6 @@ class EditExpStatePropertyCardIsCheckpointCmd(ExplorationChange):
old_value: bool


class EditExpStatePropertyRecordedVoiceoversCmd(ExplorationChange):
"""Class representing the ExplorationChange's
CMD_EDIT_STATE_PROPERTY command with
STATE_PROPERTY_RECORDED_VOICEOVERS as allowed value.
"""

property_name: Literal['recorded_voiceovers']
state_name: str
new_value: state_domain.RecordedVoiceoversDict
old_value: state_domain.RecordedVoiceoversDict


class EditExpStatePropertyInteractionIdCmd(ExplorationChange):
"""Class representing the ExplorationChange's
CMD_EDIT_STATE_PROPERTY command with
Expand Down Expand Up @@ -1564,10 +1550,6 @@ def from_dict(
[state_domain.Hint.from_dict(h) for h in idict['hints']],
solution)

state.recorded_voiceovers = (
state_domain.RecordedVoiceovers.from_dict(
sdict['recorded_voiceovers']))

state.linked_skill_id = sdict['linked_skill_id']

state.solicit_answer_details = sdict['solicit_answer_details']
Expand Down Expand Up @@ -5165,7 +5147,7 @@ def update_states_from_model(
# incompatible changes are made to the exploration schema in the YAML
# definitions, this version number must be changed and a migration process
# put in place.
CURRENT_EXP_SCHEMA_VERSION = 60
CURRENT_EXP_SCHEMA_VERSION = 61
EARLIEST_SUPPORTED_EXP_SCHEMA_VERSION = 46

@classmethod
Expand Down Expand Up @@ -5529,11 +5511,11 @@ def _convert_v60_dict_to_v61_dict(
dict. The dict representation of the Exploration domain object,
following schema version v61.
"""
exploration_dict['schema_version'] = 60
exploration_dict['schema_version'] = 61

exploration_dict['states'] = cls._convert_states_v55_dict_to_v56_dict(
exploration_dict['states'])
exploration_dict['states_schema_version'] = 61
exploration_dict['states_schema_version'] = 56

return exploration_dict

Expand Down Expand Up @@ -6188,7 +6170,6 @@ class ExplorationChangeMergeVerifier:
# cust args changes.
PROPERTIES_CONFLICTING_CUST_ARGS_CHANGES: List[str] = [
STATE_PROPERTY_INTERACTION_SOLUTION,
STATE_PROPERTY_RECORDED_VOICEOVERS,
STATE_PROPERTY_INTERACTION_ANSWER_GROUPS
]

Expand All @@ -6200,7 +6181,6 @@ class ExplorationChangeMergeVerifier:
# answer groups changes.
PROPERTIES_CONFLICTING_ANSWER_GROUPS_CHANGES: List[str] = [
STATE_PROPERTY_INTERACTION_SOLUTION,
STATE_PROPERTY_RECORDED_VOICEOVERS,
STATE_PROPERTY_INTERACTION_CUST_ARGS
]

Expand All @@ -6212,7 +6192,6 @@ class ExplorationChangeMergeVerifier:
# solution changes.
PROPERTIES_CONFLICTING_SOLUTION_CHANGES: List[str] = [
STATE_PROPERTY_INTERACTION_ANSWER_GROUPS,
STATE_PROPERTY_RECORDED_VOICEOVERS,
STATE_PROPERTY_INTERACTION_CUST_ARGS
]

Expand Down Expand Up @@ -6374,17 +6353,6 @@ def is_change_list_mergeable(
current_exp_states = (
current_exploration.states[state_name])
if (change.property_name ==
STATE_PROPERTY_CONTENT):
if (old_exp_states.content.html ==
current_exp_states.content.html):
if (STATE_PROPERTY_CONTENT not in
self.changed_translations[state_name] and
STATE_PROPERTY_RECORDED_VOICEOVERS not in
self.changed_properties[state_name]):
change_is_mergeable = True
if not self.changed_properties[state_name]:
change_is_mergeable = True
elif (change.property_name ==
STATE_PROPERTY_INTERACTION_ID):
if (old_exp_states.interaction.id ==
current_exp_states.interaction.id):
Expand Down Expand Up @@ -6480,14 +6448,6 @@ def is_change_list_mergeable(
change_is_mergeable = True
if not self.changed_properties[state_name]:
change_is_mergeable = True
elif (change.property_name ==
STATE_PROPERTY_RECORDED_VOICEOVERS):
if not self.changed_properties[state_name].intersection(
self.PROPERTIES_CONFLICTING_VOICEOVERS_CHANGES +
[STATE_PROPERTY_RECORDED_VOICEOVERS]):
change_is_mergeable = True
if not self.changed_properties[state_name]:
change_is_mergeable = True
elif change.cmd == CMD_EDIT_EXPLORATION_PROPERTY:
change_is_mergeable = (
getattr(exp_at_change_list_version, change.property_name)
Expand Down
Loading