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

Create theAthleteInsider.yml #5033

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

ArjaynP
Copy link

@ArjaynP ArjaynP commented Jan 19, 2025

The Athlete Insider is an all-in-one daily sports news website that I created myself. It delivers the latest coverage and detailed analysis on major sports storylines from the NBA, NFL, UEFA Champions League, and for a limited time the Paris 2024 Olympics. For each league, I also included game breakdown analysis, player statistics, standings, and betting odds. To create this website, I used HTML, CSS, JavaScript, JQuery, and an API implementation from Rapid API to display the NBA conference standings. I called this API using Javascript as well, and fetch as the client.

In both the nbaapi.js and nbaapiwest.js files, you will notice that I did not include the API key and simply wrote "ENTER API KEY HERE." If you would like to test out this program to see how this performs, you would need to register for free on RapidAPI to have access. Once you're registered, you can search for "API-NBA." It should allow you to subscribe to the API for free using the "Free Plan", as long as you do not go over the quota limits. For this API, it should be 100 objects per day, but make sure you confirm this under the subscription info. Once you're subscribed, on the left side, click any of the endpoints. This doesn't matter since you're only looking for the API key. At this point, you should be able to see your API key. Copy and paste it on both nbaapi.js, nbaapiwest.js files where it says "ENTER API KEY HERE." Now, you should be able to run the website through a live server and view the NBA conference standings provided by "API-NBA."

@shiftbot
Copy link
Contributor

👋 I'm a robot checking the state of this pull request to save the human reviewers time. I noticed this PR added or modififed the data files under _data/projects/ so I had a look at what's changed.

As you make changes to this pull request, I'll re-run these checks.

_data/projects/theAthleteInsider.yml

I had some troubles parsing the project file, or there were fields that are missing that I need.

Here's the details:

  • Tag 'HTML' contains invalid characters. Allowed characters: a-z, 0-9, +, #, . or -. Spaces are not allowed.
  • Tag 'CSS' contains invalid characters. Allowed characters: a-z, 0-9, +, #, . or -. Spaces are not allowed.
  • Tag 'JavaScript' contains invalid characters. Allowed characters: a-z, 0-9, +, #, . or -. Spaces are not allowed.
  • Tag 'JQuery' contains invalid characters. Allowed characters: a-z, 0-9, +, #, . or -. Spaces are not allowed.

@shiftbot shiftbot added the needs-work Reviewer has indicated changes need to be make to this pull request before it can be merged label Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-work Reviewer has indicated changes need to be make to this pull request before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants