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.

Sign up for GitHub

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

Jump to bottom

new feature: weibo poller #63

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

new feature: weibo poller #63

wants to merge 14 commits into from

Conversation

zzbslayer
Copy link

国服官方微博推送。目前只推送文字。视频、图片暂未处理。

@zzbslayer
Copy link
Author

图片推送。视频仅发送链接。

Ice9Coffee
Ice9Coffee requested changes May 8, 2020
View reviewed changes
Copy link
Owner

@Ice9Coffee Ice9Coffee left a comment

Choose a reason for hiding this comment

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

Plz read code review comments.

hoshino/res.py Outdated Show resolved Hide resolved
hoshino/modules/weibo/weibo.py Outdated Show resolved Hide resolved
hoshino/modules/weibo/weibo.py Outdated Show resolved Hide resolved
hoshino/modules/weibo/weibo.py Show resolved Hide resolved
hoshino/modules/weibo/weibo.py Outdated Show resolved Hide resolved
Ice9Coffee
Ice9Coffee requested changes May 8, 2020
View reviewed changes
hoshino/modules/weibo/weibo.py Outdated Show resolved Hide resolved
hoshino/res.py Outdated Show resolved Hide resolved
@zzbslayer
Copy link
Author

已在最新 commit 中按照 code review 更改。
2c0abae

Ice9Coffee
Ice9Coffee requested changes May 8, 2020
View reviewed changes
hoshino/modules/weibo/weibo.py Show resolved Hide resolved
hoshino/res.py Outdated Show resolved Hide resolved
Ice9Coffee
Ice9Coffee reviewed May 9, 2020
View reviewed changes
hoshino/res.py Outdated Show resolved Hide resolved
Ice9Coffee
Ice9Coffee requested changes May 9, 2020
View reviewed changes
hoshino/modules/weibo/weibo.py Outdated Show resolved Hide resolved
Ice9Coffee
Ice9Coffee previously approved these changes May 9, 2020
View reviewed changes
@Ice9Coffee Ice9Coffee mentioned this pull request May 10, 2020
Closed
@zzbslayer
Copy link
Author

之前在推送转发的微博时，bot 仅仅发送转发栏的内容，不发送原博内容。
最新的 commit 中，bot 发送转发栏的内容以及原博内容。

@zzbslayer
Copy link
Author

在配置文件中为微博账户配置别名，用户可以根据别名主动的获取指定微博账户的最新微博（至多五条）。

# config.json
[{
    "service_name": "bcr-weibo",
    "enable_on_default": true,
    "users":[{
        "user_id": "6603867494",
        "alias": ["公主连接", "公主连结", "公主链接"],   # 微博账户别名
        "filter": true
    }]
    
}]

# qq群聊
@bot 看微博 公主连接

@zzbslayer
Copy link
Author

启动时初始化微博缓存；修正了微博正文中网页链接的处理逻辑。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ice9Coffee Ice9Coffee Ice9Coffee left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@zzbslayer @Ice9Coffee