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

add rss feed for mirror #8641

Closed
wants to merge 2 commits into from
Closed

add rss feed for mirror #8641

wants to merge 2 commits into from

Conversation

fifteen42
Copy link

该 PR 相关 Issue / Involved issue

#8567

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

/mirror/entries

新RSS检查列表 / New RSS Script Checklist

  • New Route
  • Documentation
    • CN
    • EN
  • 全文获取 fulltext
    • Use Cache
  • 反爬/频率限制 anti-bot or rate limit?
    • 如果有, 是否有对应的措施? If yes, do your code reflect this sign?
  • 日期和时间 date and time
    • 可以解析 Parsed
    • 时区调整 Correct TimeZone
  • 添加了新的包 New package added
  • Puppeteer

说明 / Note

@vercel
Copy link

vercel bot commented Dec 2, 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/HRerQ84Xz8dmFj6oNnfjYxF3wDAw
✅ Preview: https://rsshub-git-fork-fifteen42-master-diy.vercel.app

Copy link
Collaborator

@TonyRL TonyRL left a comment

Choose a reason for hiding this comment

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

  1. Use Close #8567 instead of mentioning it.
  2. Please use v2 script standard v2 路由规范 #8236


<Route author="fifteen42" example="/mirror/tingfei.eth" path="/mirror/:id" :paramsDesc="['user id']" />

###
Copy link
Contributor

Choose a reason for hiding this comment

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

Why there is an empty title?

Copy link
Author

Choose a reason for hiding this comment

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

It is useless, I make a mistake.

lib/routes/mirror/entries.js Outdated Show resolved Hide resolved
Comment on lines +29 to +33
title: item.find('.css-1b1esvm').first().text(),
author: item.find('.css-1sdf1k3').first().text(),
description: item.find('.css-1koo5r3').text(),
pubDate: item.find('.css-qng491').text(),
link: link + item.find('.css-cts56n a').attr('href'),
Copy link
Contributor

@SukkaW SukkaW Dec 2, 2021

Choose a reason for hiding this comment

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

It seems that the website is using the CSS-in-JS approach, which means the className can be random.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for remanding, I tested some pages and It works, I will try to find another way to get the content if it don't work.

Co-authored-by: Sukka <isukkaw@gmail.com>
@SettingDust
Copy link
Contributor

The route path is a link for test, which can access by preview. In your situation, it's https://rsshub-git-fork-fifteen42-master-diy.vercel.app/mirror/entries. But it will return 404.

@wong2
Copy link

wong2 commented Dec 7, 2021

@rde9
Copy link
Contributor

rde9 commented Dec 18, 2021

The article link for *.eth should be https://mirror.xyz/*.eth/contentId rather than https://mirror.xyz/*.eth/*.eth/contentId as shown in the rss feed. (this problem is also found in 0x addresses)

RSS feed for *.mirror.xyz works properly.

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.

None yet

6 participants