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

fix(twitter): set title to author #17671

Closed
wants to merge 1 commit into from
Closed

Conversation

ricklahaye
Copy link
Contributor

@ricklahaye ricklahaye commented Nov 21, 2024

Example for the Proposed Route(s) / 路由地址示例

New RSS Route Checklist / 新 RSS 路由检查表

  • New Route / 新的路由
  • Anti-bot or rate limit / 反爬/频率限制
    • If yes, do your code reflect this sign? / 如果有, 是否有对应的措施?
  • Date and time / 日期和时间
    • Parsed / 可以解析
    • Correct time zone / 时区正确
  • New package added / 添加了新的包
  • Puppeteer

Note / 说明

Hi All.

I am not too familiar with JS so apologies if it does not match full standards, but tested it and seem to work. Is related to discussion #17668

This PR allows title to be set to author only. This as Twitter does not have titles, and uses description now; which does not look great when RSS client shows preview of first few lines (which should often be enough to get grasp of the tweet).

@github-actions github-actions bot added the Route label Nov 21, 2024
Copy link
Contributor

Auto Route Test failed, please check your PR body format and reopen pull request. Check logs for more details.
自动路由测试失败,请确认 PR 正文部分符合格式规范并重新开启,详情请检查 日志

@github-actions github-actions bot added the Auto: Route No Found Automatic test failed to test route in PR body label Nov 21, 2024
@github-actions github-actions bot closed this Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto: Route No Found Automatic test failed to test route in PR body Auto: Route Test Failed Route
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant