Skip to content

Make this thing roughly work again#9

Open
rjmunro wants to merge 4 commits intoNathanJPhillips:masterfrom
rjmunro:rjmunro/fixes
Open

Make this thing roughly work again#9
rjmunro wants to merge 4 commits intoNathanJPhillips:masterfrom
rjmunro:rjmunro/fixes

Conversation

@rjmunro
Copy link
Contributor

@rjmunro rjmunro commented Nov 28, 2025

Fixes:

Fixed the extraction of the next page URL in the GitHub API pagination logic in SearchResults.js

This was preventing it working

Removed Travis CI integration

Travis isn't used any more. Also it required configuring with a manually created token, there was non anonymous mode like with the GitHub API.

Disabled automatic check for updates

It was causing errors with the github api rate limits and used window.close to reload, which doesn't work in a normal web page.

Feature:

Added a new GitHub Actions workflow in .github/workflows/deploy.yml to automatically deploy the site to GitHub Pages

So you won't have to install it, it's just available to use.

It was getting 403 forbidden errors due to rate limit exceeded, even
though the other API request was working. I guess there are different
buckets of rate limits for different APIs.
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.

1 participant