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.

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

Rename DUBBING_ORIGINAL_DIALOGUE_LIST and DUBBING_TRANSLATED_DIALOGUE_LIST #102

Closed
wants to merge 1 commit into from

Conversation

nigelmegitt
Copy link
Contributor

@nigelmegitt nigelmegitt commented Dec 16, 2022

Closes #87.

  • Change DUBBING_ORIGINAL_DIALOGUE_LIST to ORIGINAL_TRANSCRIPT and change the name "Original Language Dialogue List" to "Original Language Transcript"
  • Change DUBBING_TRANSLATED_DIALOGUE_LIST to TRANSLATED_TRANSCRIPT and change the name "Translated Dialogue List" to "Translated Transcript"
  • Split out the list of script types that are Dubbing Scripts into a separate paragraph for readability.
  • Propagate changes throughout the examples
  • Add an Ed Note before Example 12 - looks like it's orphaned and needs to move higher up, possibly.

Preview | Diff

…_LIST

Closes #87.

* Change `DUBBING_ORIGINAL_DIALOGUE_LIST` to `ORIGINAL_TRANSCRIPT` and change the name "Original Language Dialogue List" to "Original Language Transcript"
* Change `DUBBING_TRANSLATED_DIALOGUE_LIST` to `TRANSLATED_TRANSCRIPT` and change the name "Translated Dialogue List" to "Translated Transcript"
* Split out the list of script types that are Dubbing Scripts into a separate paragraph for readability.
* Propagate changes throughout the examples
* Add an Ed Note before Example 12 - looks like it's orphaned and needs to move higher up, possibly.
@nigelmegitt nigelmegitt self-assigned this Dec 16, 2022
Copy link
Contributor

@cconcolato cconcolato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we simply use "Script" instead of "Transcript"? At least for consistency with the other values.

@nigelmegitt
Copy link
Contributor Author

Can we simply use "Script" instead of "Transcript"? At least for consistency with the other values.

I'd rather not - see also #101 where I try to make the distinction clearer and more useful.

@nigelmegitt
Copy link
Contributor Author

This will have conflicts with #101 but is broadly aligned with the direction of change in that pull request too.

@nigelmegitt nigelmegitt mentioned this pull request Dec 23, 2022
@nigelmegitt
Copy link
Contributor Author

I've merged this into #101, so closing this. Makes sense to review them together in the context of improving the consistency of how the spec uses the terminology "script" and "transcript".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename DUBBING_ORIGINAL_DIALOGUE_LIST and DUBBING_TRANSLATED_DIALOGUE_LIST
2 participants