-
Notifications
You must be signed in to change notification settings - Fork 17
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
Feature/Release Notes - Added to Home Page under What's New #26
base: main
Are you sure you want to change the base?
Conversation
@Wright4i Interesting. Why is the token required? Can you not access the public API for released without a token? |
@worksofliam You'd think, but I tried and it didn't work. Guessing so they can track you for rate-limiting/abuse purposes. |
@Wright4i @worksofliam Shouldn't we try and complete this? Would be great to have in the docs... 😍 |
@chrjorgensen you're right! I completely forgot this was out here open. @worksofliam any issues with the API key usage? If not I can address the conflicts and we can get this merged in. |
@Wright4i No problem with token usage! This would be cool to have. |
Link to demo site: https://wright4i.github.io/docs/
Uses GitHub API to grab details of a public repo (vscode-ibmi)
Home Page
Release Notes
githubAPI.ts
'sfetchFromGithub()
has a defaultlimit: number = 5
so we can adjust to taste.Packages added to project
Server side only, no performance impact
dotenv
Reason - Best practices. Needed
.env
files forAPI_GITHUB_TOKEN
. Created.env.example
for local development.marked
Reason - Release notes from API was in markdown. marked converted to HTML so I could set the
innerHTML
of the astro component.Tasks to Build on Merge
public_repo
scopeAPI_GITHUB_TOKEN
.github/workflows/astro.yml
was updated to importAPI_GITHUB_TOKEN
vscode-ibmi
to trigger build ondocs
soRelease Notes
are always current