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
Update Github trending widget #1562
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for dashy-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @alhazmy13 - looks good :)
Just a small thing, make sure the lint checks pass, e.g. single quotes
@@ -77,11 +77,11 @@ export default { | |||
slug: repo.repo, | |||
desc: repo.desc, | |||
lang: repo.lang, | |||
link: repo.repo_link, | |||
link: "https://github.com" + repo.repo, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Double quotes will fail the linter. Also, we can can use string interpolation.
- link: "https://github.com" + repo.repo,
+ link: `https://github.com/${repo.repo}`,
Thank you for contributing to Dashy! So that your PR can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category:
Update the widget URL
Overview
This pull request updates the URL for the GitHub trending widget. It changes from the forked repository (https://github.com/Lissy93/gh-trending-no-cors) to the original repository (https://github.com/doforce/github-trending). This update is necessary because the application deployed from the forked repository (https://gh-trending-repos.herokuapp.com/repo) is no longer available, with this change we do update the widget UI & Paramter to match the new response schema for the project.
Code Quality Checklist (Please complete)