Skip to content

add accent options for nan-tw #3473

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

Merged
merged 1 commit into from
Apr 5, 2022
Merged

add accent options for nan-tw #3473

merged 1 commit into from
Apr 5, 2022

Conversation

irvin
Copy link
Member

@irvin irvin commented Feb 19, 2022

Pull Request Form

Type of Pull Request

  • Other

Add accents data for nan-tw. We use the same list at zh-tw which are the birthplaces (city) as the default options.

This is the suggest options from local linguistic researchers, the reason is that people often don't know the formal name or correct accents (or in Mandarin cases, people don't know there are accents and differences within Taiwan Mandarin), so the researchers use birthplace to distinguish and do analysis.

Acknowledging contributors

@zcolleyz
Copy link
Contributor

@Heyhillary @ftyers what are the next steps for this pull-request?

@Heyhillary
Copy link
Contributor

Hey @zaccolley , before we submit the merge - can we check-in with EM as there is going to be some follow up work with the accent capture write-ins to update the default accent lists.

@Heyhillary Heyhillary self-assigned this Mar 23, 2022
@zcolleyz
Copy link
Contributor

zcolleyz commented Apr 5, 2022

@Heyhillary got the go ahead from EM merging this. Thank you @irvin

@zcolleyz zcolleyz merged commit 496a200 into common-voice:main Apr 5, 2022
@irvin irvin deleted the main branch April 5, 2022 13:29
@irvin
Copy link
Member Author

irvin commented Apr 5, 2022

thanks!

@irvin
Copy link
Member Author

irvin commented May 17, 2022

@zaccolley hi, this still not showing up in the accent dropdown menu. Has it been deployed to the production site?

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.

3 participants