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

Added API request for list of items in toolbox #75

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

emergenitro
Copy link

This is to give you a list of items from manifest.js when you visit https://toolbox.hackclub.com/api/list.

This was done upon request on Slack.

Copy link

vercel bot commented Nov 18, 2024

@emergenitro is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Nov 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
toolbox ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 20, 2024 9:40pm

@lachlanjc
Copy link
Member

📝 Changed routes:

Commit 773d8bb (https://toolbox-amvgfyfly.hackclub.dev).

@BrightTheBackpack
Copy link
Member

Should it add the stuff from here?
https://github.com/hackclub/toolbox/blob/main/lib/github.js

@emergenitro
Copy link
Author

I don't think so. Should it?

If you want me to, I can add that in as well but I'm not too sure what exactly I could add from it

@BrightTheBackpack
Copy link
Member

everything in the "CURRENT" section of teh toolbox comes from https://github.com/hackclub/site/blob/main/lib/carousel.json, which is what github.js and content.js pull from. If we want the API to include everything from the toolbox, it should include that.

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.

3 participants