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

Feature geo daily photo #7554

Closed
wants to merge 13 commits into from
Closed

Feature geo daily photo #7554

wants to merge 13 commits into from

Conversation

OrangeEd1t
Copy link

@OrangeEd1t OrangeEd1t commented May 20, 2021

该 PR 相关 Issue / Involved issue

Close #7471

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

/natgeo/dailyphoto

新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)
  • 目标是否有提供日期? Is there a date in the source?
    • 如果有,包是否正确解析? If there is, can this script provide this info?
    • 如果有提供解析能力,时区是否正确调整? Is the timezone correctly provided?
    • 如果有提供日期,但是没有提供解析,请说明原因 If there is a date but this script does not parse, please provide your reason.
  • 是否引入的新的包? Any new package introduced?
    • 如果有, 请说明原因. If yes, please state your reason
  • 是否使用了Puppeteer? Make use of Puppeteer?
    • 如果有, 请说明原因. If yes, please state your reason

说明 / Note

  • 因为需要将网页渲染完成后才能获取到图片地址,所以使用的是【使用puppeteer渲染页面获取数据】的方式来获取数据。
  • 因为网页面很大,我并未将整个网页都获取回来进行解析,仅是截取了网页中包含了每日一图的相关数据的html代码。

@vercel
Copy link

vercel bot commented May 20, 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/8CvvvL7Ab7mYFjMzSEoAmYDvC2u5
✅ Preview: https://rsshub-do-not-use-git-fork-orangeed1t-feature-geodailyphoto-diy.vercel.app

@github-actions
Copy link
Contributor

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

@github-actions github-actions bot added the Auto: Route No Found Automatic test failed to test route in PR body label May 20, 2021
@github-actions github-actions bot closed this May 20, 2021
@OrangeEd1t OrangeEd1t deleted the feature-geoDailyPhoto branch May 20, 2021 06:17
@OrangeEd1t OrangeEd1t restored the feature-geoDailyPhoto branch May 20, 2021 06:17
@OrangeEd1t OrangeEd1t deleted the feature-geoDailyPhoto branch May 20, 2021 06:19
@OrangeEd1t OrangeEd1t restored the feature-geoDailyPhoto branch May 20, 2021 06:21
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] 国家地理-每日一图 获取失败
1 participant