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

统一黑白名单,支持规则优先级、搜索、多选管理 #84

Closed
wants to merge 3 commits into from

Conversation

Hovn
Copy link

@Hovn Hovn commented Dec 12, 2021

【目标】
被拦截的短信很多时候我们也希望能进行方便的管理,比如进行查询,删除某些特定的短信。因此对相关功能进行了补充。

【主要更新点】
统一黑白名单,支持规则优先级 (fixes #54 )
支持按发送者或消息内容进行搜索
新增多选管理(删除、恢复、已读)
显示接收SIM卡(subid)
规则及短信调整为多行显示(在列表界面即可完整查看)(fixes #82 )
识别信息中网址,链接可点击跳转

Hovn added 3 commits December 12, 2021 08:13
支持搜索
多选管理(删除、恢复、已读)
显示接收SIM卡(subid)
多行显示
识别信息中网址
之前使用changeCursor(),在一些场景下,界面的数据不会自动更新(如查询后回到主界面)
@Hovn Hovn changed the title 统一黑白名单,支持规则优先级 统一黑白名单,支持规则优先级、搜索、多选管理 Dec 12, 2021
Copy link
Owner

@apsun apsun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thanks for the PR! I love the idea of these features, I would have implemented them myself but never had the time. That said, this change is 1.5k LoC and touches 40 code files, which makes it quite difficult to review. Could you please split up the PR into smaller, separate PRs (one for each feature)? That will make the code much easier to understand. Thanks!

@Hovn
Copy link
Author

Hovn commented Dec 13, 2021

这些代码也是多年使用NekoSMS,因为自己的需求而积累下来的更改。所以影响范围较大。
我将尝试拆分成更小的PR,不过这可能也需要一些时间,因为时间也不富裕。
感谢~

@Hovn
Copy link
Author

Hovn commented Jul 16, 2022

因累计改动较多,待后续分解后重新PR

@Hovn Hovn closed this Jul 16, 2022
@Hovn Hovn deleted the dev branch July 17, 2022 08:12
@Hovn Hovn restored the dev branch July 17, 2022 08:13
@Hovn Hovn deleted the dev branch July 17, 2022 08:13
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.

Change the expression input box to a multi-line input box Priority support.
2 participants