add APIs for getting toolbox items#271
Conversation
|
@Mahmoud1205 is attempting to deploy a commit to the Hack Club Team on Vercel. A member of the Team first needs to authorize it. |
|
hey, thanks for the pr! before this can be merged there are a few things that need to be fixed though: the category IDs aren't stable - you're using the array index as the ID, so if anyone reorders the manifest the IDs shift and
file naming - Next.js API routes are conventionally lowercase or kebab-case, not PascalCase. the yarn.lock diff is huge (+360/-695) - can you explain what changed there? dependency changes need to be intentional, not a side effect of running install with a different node version. brace style is inconsistent between the two files. pick one and stick to it. great work, but it just needs a bit of cleanup before it's ready to merge! i've marked it as a draft for now :) |
fixes #190
there are two new APIs: GetCategories which returns a list of categories with their names and IDs.
and GetItems which returns a list of items from a category ID. or all the categories if a category ID wasn't selected
the API is very simple but i will upgrade it soon if this PR gets merged.