next-theme / hexo-theme-next

🎉 Elegant and powerful theme for Hexo.
https://theme-next.js.org
Other
2.32k stars 416 forks source link

Update beian URL #723

Closed Leao9203 closed 6 months ago

Leao9203 commented 6 months ago

PR Checklist

PR Type

What is the current behavior?

Issue resolved: The link of the MPS public query has been changed. This PR has changed the link of gongan_num to the new.

welcome[bot] commented 6 months ago

Thanks so much for opening your first PR here!

CLAassistant commented 6 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

stevenjoezhang commented 6 months ago

_config.yml里面的http://www.beian.gov.cn也可以替换为https://beian.mps.gov.cn,然后http://www.beian.gov.cn/portal/download这个下载链接也可以删掉(或者看看有没有新的官方下载链接?)

Leao9203 commented 6 months ago

_config.yml里面的http://www.beian.gov.cn也可以替换为https://beian.mps.gov.cn,然后http://www.beian.gov.cn/portal/download这个下载链接也可以删掉(或者看看有没有新的官方下载链接?)

下载链接的话,目前好像是需要登录到 MPS 上才可以下载,而且似乎并不提供直接的 http 下载链接,最终是以一个 blob: 开头的链接保存的。可以直接把链接修改为 https://beian.mps.gov.cn/web/business/businessHome/website 吗?

github-actions[bot] commented 6 months ago

This pull request contains changes to the configuration file. Please make sure the documentation in NexT website is changed or added. Please edit relevant source files here: https://github.com/next-theme/theme-next-docs/tree/master/source/docs and create a pull request with the changes here: https://github.com/next-theme/theme-next-docs/pulls

stevenjoezhang commented 6 months ago

https://beian.mps.gov.cn/web/business/businessHome/website 这个没登录的话好像会重定向到首页,还是直接删掉好了

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 7035386669

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report. To ensure accuracy in future PRs, please see these guidelines. A quick fix for this PR: rebase it; your next report should be accurate.


Totals Coverage Status
Change from base Build 6458994765: 0.0%
Covered Lines: 394
Relevant Lines: 399

💛 - Coveralls
Leao9203 commented 6 months ago

https://beian.mps.gov.cn/web/business/businessHome/website 这个没登录的话好像会重定向到首页,还是直接删掉好了

我是修改成这个样子了 图片

stevenjoezhang commented 6 months ago

Thanks!

Leao9203 commented 6 months ago

Thanks!

It was nothing!

welcome[bot] commented 6 months ago

Congrats on merging your first pull request here! :tada: How awesome!