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

vyextrahovane texty od MZCR - chybi cz_transcription #46

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

romanzdk
Copy link
Contributor

chybi cz_transcription

@JuroUhlar
Copy link
Collaborator

@met What about these phrases? We can generate transcriptions automatically now, so we can easily include them.

@met
Copy link
Collaborator

met commented Apr 14, 2022

@JuroUhlar Plesase wait with this for now.

We need to create something called "external dictionary catergory" that is handled a little bit differently from our "normal" categories. This is first, there could be more. Its content that is not ours, but from external source and we only put i on our website with the permission.

It should work pretty similar like normal category, with some smaller differences that we still need to discuss. Better wait for that after transforming Movapp into mutlilanguages website.

I noted it into trello.

@mvidner
Copy link

mvidner commented Jun 24, 2022

Ahoj @met,

I wanted to help by adding cz_transcription but it seems there is some categorization task to do, not mentioned in the PR description nor in the Trello card.

Anyway, the transcriptions are independent of categorization, right?

@met
Copy link
Collaborator

met commented Jun 24, 2022

Ahoj @met,

I wanted to help by adding cz_transcription but it seems there is some

Hi @mvidner
Some of these information are old. We generate transcription automatically now. No need to make it manually.

This PR waits only for "categorization" that is not ready yet. And because we are changing our data source (see https://github.com/cesko-digital/movapp-data for more) it is probably better wait after this change is made.

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.

4 participants