-
Notifications
You must be signed in to change notification settings - Fork 4
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
Translations #102
Translations #102
Conversation
… into translations
+ add elasticsearch & pgsync
merge main
Translations
docker/docker-compose.build.yml
Outdated
- REPCO_ADMIN_TOKEN=kUBY0zsPHC9ubj3T6DZJKUACi3M= | ||
- REPCO_URL=http://localhost:8765 | ||
- CBA_API_KEY=k8WHfNbal0rjIs2f | ||
- AP_BASE_URL=http://localhost:8765/ap |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The AP_BASE_URL
can be removed now that #101 is merged
@@ -591,9 +611,14 @@ export class ActivityPubDataSource | |||
subtitleEntities && entities.push(...subtitleEntities) | |||
} | |||
|
|||
var titleJson: { [k: string]: any } = {} | |||
titleJson[video.language?.name || 'de'] = { value: video.name } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ingo reported a bug about English
appearing as an object key, where it should be en
. I suspect this line being the cause.
I also noticed an error: Title contains e.g. the word “English” as an object key. There should be a two-digit language code “en” in there.
"title": {
"English": {
"value": "A step to the left, a step to the right: the new European centre"
}
reworked db schema for multilingual support