tim-hub / obsidian-bible-reference

Take Bible Study notes easily in the popular note-taking app Obsidian, with automatic verse and reference suggestions.
https://antioch.tech/obsidian-bible-reference/
MIT License
239 stars 42 forks source link

Show internationalized book name #134

Closed kailichiang closed 8 months ago

kailichiang commented 1 year ago

Is your feature request related to a problem? Please describe. When I set the default bible version to non-English version, I'd like to see the correspoding book name. For example, I set it as "Chinese - Chinese Union Version (Traditional) @Bolls Life," When I type --genesis 2:3, instead of showing Genesis 2:3 - CUV, I'd like to see it as 創世紀 2:3 - CUV or 創 2-3 (this is the short name of Genesis in Chinese).

Describe the solution you'd like Instead of showing English book name, show book name in corresponding language.

Describe alternatives you've considered Not yet know. Probably one more option to set language?

Additional context

Xnip2023-09-13_14-55-19
tim-hub commented 1 year ago

Thanks this is a know limitation.

Currently I am working on it to bring more language book names.

Would you want to help for some translation or translation review work? If yes, I could forward you the link for it. 😁

kailichiang commented 1 year ago

Sure. Plz forward me it.

tim-hub commented 1 year ago

Currently the trigger of book names could be done in more langauges in 2.1.1

For example, this Italian version

https://github.com/tim-hub/obsidian-bible-reference/releases/tag/2.1.1

tim-hub commented 1 year ago

Sure. Plz forward me it.

Hi @kailichiang

This is the repo to have different translations,

ref: https://github.com/AntiochTechnology/bible-reference-toolkit/tree/master/packages/bible-book-names-intl/src/data/translations

It is a JSON file in a spercific structure, there are 3 langauges there, if you can help for Chinese, that will be great

feel free to ping me if need help

tim-hub commented 1 year ago

In version 2.2.0

https://github.com/tim-hub/obsidian-bible-reference/releases/tag/2.2.0

Multiple languages are supported, Spanish, Italian, Japanese.

However it is not that easy to support all langauges.

grnsmn commented 1 year ago

Hi @tim-hub , I'm interested in helping to add translation support for the reference header and solve this issue. However, after inspecting the code, I still can't figure out how this is generated.

While I see that the plugin fetches translation references from the library at the following link: https://github.com/AntiochTechnology/bible-reference-toolkit/tree/master/packages/bible-book-names-intl, I'm still unclear on how it doesn't seem to translate the header into the user's chosen language.

Could you provide more information or guidance on how the translation for the reference header can be implemented? This would help me better understand how to contribute effectively.

Thank you!

tim-hub commented 1 year ago

Hi @tim-hub , I'm interested in helping to add translation support for the reference header and solve this issue. However, after inspecting the code, I still can't figure out how this is generated.

While I see that the plugin fetches translation references from the library at the following link: https://github.com/AntiochTechnology/bible-reference-toolkit/tree/master/packages/bible-book-names-intl, I'm still unclear on how it doesn't seem to translate the header into the user's chosen language.

Could you provide more information or guidance on how the translation for the reference header can be implemented? This would help me better understand how to contribute effectively.

Thank you!

Uh, currently, the package was used only for match the book name.

If want to show it in header (rendering), here need to be changed. https://github.com/tim-hub/obsidian-bible-reference/blob/0ac39dd6d50129d73d00e9583d96c1859de7a4d0/src/provider/BaseBibleAPIProvider.ts#L47C1-L54C4

to use the package bible-book-names-intl

But careful, since it is a base class.

stale[bot] commented 10 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 8 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.