Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

add Simplified Chinese Localization #17

Merged
merged 4 commits into from
Aug 9, 2024
Merged

add Simplified Chinese Localization #17

merged 4 commits into from
Aug 9, 2024

Conversation

L1cardo
Copy link
Contributor

@L1cardo L1cardo commented Aug 9, 2024

Description

  1. add Simplified Chinese Localization
  2. add "Run App" button
  3. refactor WelcomView
    image

Linked Issues

Additional context

@QiuChenly
Copy link
Collaborator

非常感谢您的提交,能否合并成一个commit呢?

@L1cardo
Copy link
Contributor Author

L1cardo commented Aug 9, 2024

3个改进点我分成 3 个 commit,方便后期有问题随时按需回退

你要 1 个 commit 的话,我晚点再搞,现在在外面

@QiuChenly
Copy link
Collaborator

了解了,那就稍等review完这几个commit结束就合并您的提交。非常感谢!

@wibus-wee
Copy link
Owner

感谢!我觉得是否应该拆分为三个pr进行提交?

我仅启动了压缩提交,这种情况下会导致这个commit同时包含了您三个修改点?

或者将此 commit message 设置为最主要的修改也可以

@QiuChenly
Copy link
Collaborator

@wibus-wee 本地同步测试patch后工作正常,可以考虑合并。

@wibus-wee wibus-wee merged commit 1eaa3d6 into wibus-wee:main Aug 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants