Skip to content

Comments

feat: user_id_list在config.json支持混合的模式#270

Merged
dataabc merged 1 commit intodataabc:masterfrom
eggachecat:master
Dec 9, 2020
Merged

feat: user_id_list在config.json支持混合的模式#270
dataabc merged 1 commit intodataabc:masterfrom
eggachecat:master

Conversation

@eggachecat
Copy link
Contributor

@eggachecat eggachecat commented Dec 9, 2020

兼容现有的模式下,支持就像「.txt」那样混合的user_id_list

{
    "user_id_list": ["1729370543", {
        "id": "1669879400",
        "since_date": "2020-11-01 18:15"
    }],
}

这样使得程序的调用的时候会更灵活一点

  • 另外清理了一下commit历史 合并成一个
  • 对于非dict保持去重的功能

@eggachecat
Copy link
Contributor Author

@dataabc 这样如何呢?
不过理论上dict也可以提供去重(id一样,merge start_date/end_date 的区间)😂

@dataabc dataabc merged commit 839cd18 into dataabc:master Dec 9, 2020
@dataabc
Copy link
Owner

dataabc commented Dec 9, 2020

已merge。

感谢贡献代码。dict去重有个问题,就是id一样,但是其它配置不同,说明使用者对该用户的配置可能有疑问,如果我们自行merge start_date/end_date,会不会让使用者困惑呢,所以可以暂时不处理。

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.

2 participants