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(route): yicai wrong pubDate #7137

Merged
merged 1 commit into from
Mar 11, 2021
Merged

fix(route): yicai wrong pubDate #7137

merged 1 commit into from
Mar 11, 2021

Conversation

queensferryme
Copy link
Contributor

该 PR 相关 Issue / Involved issue

Close #6795

完整路由地址 / Example for the proposed route(s)

/yicai/brief

新RSS检查列表 / New RSS Script Checklist

  • 这个PR中包含了新的路由吗? Does this PR add new route?
    • 如果有, 请完成检查列表. If yes, please finish the check list
    • 如果你的PR符合下方某个事项, 也请注明. If any of the checklist item meets your PR, please fill it out.
    • <- 这样打勾
  • 是否提供了文档? Documentation provided?
    • 是否提供了英文文档? EN Documentation provided?
  • 是否支持全文获取? Is this RSS Script support fulltext?
    • 如果全文获取中需要访问文章链接, 是否使用了缓存? If fulltext requires to fetch detail pages, is cache used in the process?
    • 缓存说明 | How to use cache
  • 目标是否有明显的反爬/频率限制? Is there any sign of anti-bot or rate limit?
    • 如果有, 是否有对应的措施? (延长缓存时间, 写文档说明, etc.) If yes, do your code reflect this sign? (e.g. write documentations, use long cache time)
  • 是否引入的新的包? Any new package introduced?
    • 如果有, 请说明原因. If yes, please state your reason
  • 是否使用了Puppeteer? Make use of Puppeteer?
    • 如果有, 请说明原因. If yes, please state your reason

说明 / Note

@vercel
Copy link

vercel bot commented Mar 8, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/diy/rsshub-do-not-use/BqTy1e6RTaDNCYT3E17inwKSrqzq
✅ Preview: https://rsshub-do-not-use-git-fork-queensferryme-fix-yicai-diy.vercel.app

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2021

Successfully generated as following:

https://rsshub-do-not-use-erbntv65s-diy.vercel.app/yicai/brief - Success
    <?xml version="1.0" encoding="UTF-8"?>
<rss  xmlns:atom="http://www.w3.org/2005/Atom" version="2.0"

>
    <channel>
        <title><![CDATA[第一财经 - 直播区]]></title>
        <link>https://www.yicai.com/brief</link>
        <atom:link href="https://rsshub-do-not-use-erbntv65s-diy.vercel.app/yicai/brief" rel="self" type="application/rss+xml" />
        <description><![CDATA[第一财经 - 直播区 - Made with love by RSSHub(https://github.com/DIYgod/RSSHub)]]></description>
        <generator>RSSHub</generator>
        <webMaster>[email protected] (DIYgod)</webMaster>
        
        
        
        <language>zh-cn</language>
        
        <lastBuildDate>Mon, 08 Mar 2021 11:15:31 GMT</lastBuildDate>
        <ttl>5</ttl>
        
        <item>
            <title><![CDATA[【今日两市共发生90宗大宗交易 机构抢筹这些个股】 今日两市共有57只个股发生90宗大宗交易,总成交额约为14.69亿元。<br /><br />从溢价率来看,当日共有9只股票...]]></title>
            <description><![CDATA[【今日两市共发生90宗大宗交易 机构抢筹这些个股】 今日两市共有57只个股发生90宗大宗交易,总成交额约为14.69亿元。<br><br>从溢价率来看,当日共有9只股票溢价成交,其中赛微电子、出版传媒、海天味业位居前3位,溢价率分别为22.2%、10.52%、5.01%。从折价率来看,居前三的股票分别是弘信电子、南华仪器、节能风电,折价率分别为21.4%、20.89%、14.43%。<br><br>从成交金额来看,排名前三的股票分别为天邦股份、华铁应急、三七互娱,成交额分别为2.38亿元、1.76亿元、1.27亿元。]]></description>
            <pubDate>Mon, 08 Mar 2021 11:10:00 GMT</pubDate>
            <guid isPermaLink="false">https://www.yicai.com/brief/100971888.html</guid>
            <link>https://www.yicai.com/brief/100971888.html</link>
            
            
            
            
            

@github-actions github-actions bot added the Auto: Route Test Complete Auto route test has finished on given PR label Mar 8, 2021
@NeverBehave NeverBehave changed the title fix: yicai wrong pubDate fix(route): yicai wrong pubDate Mar 11, 2021
@NeverBehave NeverBehave merged commit e673434 into DIYgod:master Mar 11, 2021
@queensferryme queensferryme deleted the fix/yicai branch March 11, 2021 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto: Route Test Complete Auto route test has finished on given PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pubDate 时间错误
2 participants