Closed Ice-Pendragon closed 2 years ago
Thanks for submitting this! This package is actually kind of deprecated, since we want to update Bastion to use more consistent data structures for card information. We'll accept this patch when it's ready though. If you know of any good Korean data sources, let us know!
Thanks for kind reply, and I'm sorry for my bad english. I wrote patch note "Korean CDB will be updated by JSY1728" as I made this regex first and told him/her to change CDB as it fits regex, and the CDB is now changed.
As I'm just a amateur, I hope you to check first with some examples before update the bot - you can check these cards:
D/D Gryphon (pendulum, tagged for search) D - Fortune (non-pendulum, tagged for search) Majespecter Cat - Nekomata (pendulum, no pendulum effect) Mild Turkey (pendulum, normal monster) Dark Doriado (pendulum)
If any of result is suspicious, don't mind to tell me that I can find the problem!
um... sounds good. any others problem?
Here's how things will go in hopefully a few hours. The creator is unavailable at the moment and also in a different timezone to update this package. When available, this pull request will be merged and she will bump this package to 1.4.0. Then the old instance of Bastion will be updated to incorporate the changes.
@JSY1728 If you have foundational source of truth for Korean translations, as you maintain multiple CDBs, and you're willing to provide that to the Bastion project, please let me know! With a consistent, normalized database, we can do cool stuff like this on the new instance.
@Ice-Pendragon also, once I get DawnbrandBots/bastion-bot#46 and DawnbrandBots/bastion-bot#47 done, if you would like to help localize Bastion's interface to Korean too, it would be much appreciated!
@JSY1728 If you have foundational source of truth for Korean translations, as you maintain multiple CDBs, and you're willing to provide that to the Bastion project, please let me know! With a consistent, normalized database, we can do cool stuff like this on the new instance.
@Ice-Pendragon also, once I get DawnbrandBots/bastion-bot#46 and DawnbrandBots/bastion-bot#47 done, if you would like to help localize Bastion's interface to Korean too, it would be much appreciated!
That would be hopeful. What strings are to be translated, and where are they used for?
What strings are to be translated, and where are they used for?
I'll describe in those issues once I build out the system for the new instance đ
Sorry for requesting again. I found an unexpected behavior. Could I change regex once more? I'm terribly sorry that I missed the problem on the screenshot.
Bug found: Last line of monster effects/lore text of Pendulum Monsters are not displayed
From:
const KOR_PEND_REG = /ă(íëëź í¨ęłź|ě¤íŹ ë°ë)ă\s*[\r\n|\r|\n](?:ěě)?([^â]*)(?:[\r\n|\r|\n]\s*â+)?/;
const KOR_MON_REG = /ă(몏ě¤í° í¨ęłź|íë ě´ë˛ í
ě¤í¸|í¨ęłź)ă\s*[\r\n|\r|\n]([^â]*)(?:[\r\n|\r|\n]\s*â?)/;
To:
const KOR_PEND_REG = /ă(íëëź í¨ęłź|ě¤íŹ ë°ë)ă\s*[\r\n|\r|\n](?:ěě)?([^â]*)(?:\s*â+)?/;
const KOR_MON_REG = /ă(몏ě¤í° í¨ęłź|íë ě´ë˛ í
ě¤í¸|í¨ęłź)ă\s*[\r\n|\r|\n]([^â]*)(?:\s*â?)/;
No need to apologize! Do you want to open another PR?
If needed, but I think regex fix above would be fine enough. Can you apply it to Galatea first and show me the result?
The result seems completely fine!
I committed this change in a0542098e537fee8458ba3f5c1b0c621243c17a0 and made it live in the old instance of Bastion.
Since JSY1728's Korean CDB includes some more informations for EDOPro users (for official Korean version of Yu-Gi-Oh! has many mistranslations, and for Korean Windows, some symbols are hard to type, and therefore hard to search), I added regex for his/her Korean CDB, and told him/her to make borderline in literalDesc 'â's from '-'s.