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

🐛 部分文字转换失败问题 #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

KimigaiiWuyi
Copy link

@KimigaiiWuyi KimigaiiWuyi commented Nov 14, 2023

当前frontend.py存在部分情况时,输入文本点击转换会直接报错 (#8),修复如下

  • 添加中文字符的排除
  • 添加空格字符、换行字符的排除
  • 将数字转为汉字再进行转换(我不清楚该项目对第三方库的态度,如果可以的话,可以在之后引用第三方库进行数字和汉字的转换)

目前仍发现问题:

  • 无法正确处理中文和英文混排情况

@KimigaiiWuyi KimigaiiWuyi mentioned this pull request Nov 14, 2023
syq163 pushed a commit that referenced this pull request Nov 17, 2023
@yihong0618
Copy link

yihong0618 commented Nov 17, 2023

@syq163 @KimigaiiWuyi @ if this is a open source project in GitHub, and this PR fix a lot of problems for the issue user use, do we need to consider community PR first, instead of using the PR method in commit, and ignore it? or you can add steps of how to make the contribute in this repo.
Thank you.

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.

None yet

2 participants