FIXED - Organise Toolbox resources into clearer categories & add new Resources / Cards - #307
Merged
Merged
Conversation
Added a new entry for 'Nest' with details about its features and usage.
Add 'Nest' entry to manifest.js
Updated icon property for CAD YSWSs entry in manifest.
Added a new entry for 'Nest Git' with relevant details.
Added Print Legion entry with details and styling.
|
@SmartSparkCoding is attempting to deploy a commit to the Hack Club Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
Author
|
Test Deployment: |
Reintroduced 'School Toolbox' with updated access for leaders.
hippogriff101
approved these changes
Aug 6, 2026
hippogriff101
left a comment
Contributor
There was a problem hiding this comment.
looks good to me (lgtm) :pf:
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (3)
manifest.js:631
- This PR description doesn’t mention changing who can access “School Toolbox”, but in this move it’s now
forUseBy: 'leaders'(it previously appeared under Resources for everyone). If the intent is only to recategorize, keep the original audience to avoid an unintended access change.
forUseBy: 'leaders'
manifest.js:418
titleColoris set toA633D6without a leading#, which is not a valid CSS color value. This will cause the FancyCard icon/title styling to render incorrectly (or fall back).
titleColor: 'A633D6',
manifest.js:638
- This link uses plain HTTP. Use HTTPS to avoid downgrade/mixed-content issues and ensure credentials/session cookies aren’t exposed in transit.
url: 'http://forms.hackclub.com/club-stickers',
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Suppressed comments (2)
manifest.js:422
- This item defines FancyCard-only styling fields (background/titleColor/descriptionColor) but is missing
fancy: true, so it will render with the plain InfoCard and those fields will be ignored. Addfancy: true(or remove the unused styling fields if it’s meant to be a plain card).
titleColor: '#A633D6',
descriptionColor: '#FFFFFF',
slack: {name: 'nest', id: 'C056WDR3MQR' },
forUseBy: 'everyone'
},
manifest.js:631
- PR description lists which leader-only resources are being moved, but this change also makes “School Toolbox” leader-only (
forUseBy: 'leaders'). If that restriction is intentional, it should be called out in the PR description; if not,forUseByshould remaineveryone.
description: 'Resources to help with school admin or IT challenges',
icon: 'briefcase',
external: true,
url: 'https://school-toolbox.hackclub.com',
forUseBy: 'leaders'
hippogriff101
approved these changes
Aug 6, 2026
hippogriff101
left a comment
Contributor
There was a problem hiding this comment.
There we go. All good.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary of Changes!
This PR slightly reorganises the Toolbox manifest.js to make resources for leaders easier to discover while adding missing 'cards' to the toolbox.
List of changes
Screenshots of Update