Fix transaction seconds and Android widget rendering - #384
Closed
quecai-niu wants to merge 1 commit into
Closed
Conversation
Owner
|
@quecai-niu 感谢贡献,还在小米真机上验证过 👍 两块改动本身方向都没问题。不过想请你先把这个 PR 拆分再继续——它现在把两个互不相关的改动和一大批格式化重排混在了一起,不好 review,将来某一块出问题也没法单独回滚。 里面其实是三类东西:
建议:
这样每个 PR 聚焦一件事,好 review,也好独立回滚。拆好之后我再逐个过、合并。辛苦 🙏 |
Author
|
感谢 review,我已经按建议拆分并去掉无关格式化重排:
两个新 PR 都基于最新
这个混合 PR 我先关闭,后续在两个拆分 PR 里继续跟进。 |
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verification