Skip to content

Conversation

@spacevini8
Copy link
Member

Ready to submit your project? Nice job! Hopefully you've read the submission guidelines. If not, make sure to read them first.

Check off the following to let the reviewers know you've done everything:
[Y] I HAVE RAN DRC ON MY DESIGN WITH ZERO ERRORS (Silkscreen errors are okay)
[Y] I HAVE BRANDED ALL MY PARTS (PCB, CASE) WITH MY HACKPAD'S NAME
[Y] I HAVE MADE SURE MY 3D PRINTED PARTS DO NOT REQUIRE SUPPORTS (Text branding overhang is okay)
[Y] I HAVE MADE SURE TO INCLUDE TOLERANCES (0.25mm)
[Y] MY HACKPAD'S PCB IS UNDER 100mm x 100mm, I DO NOT HAVE MORE THAN 16X SWITCHES, 2X ROTARY ENCODERS, 1X OLED, 16X LEDS.
[Y] IN MY FOLDER, THERE IS A README.MD FILE THAT CONTAINS THE FOLLOWING INFORMATION:

  • Full render including all the parts
  • A few sentences about the inspiration and challenges
  • BOM
  • Photos of the Schematic, PCB, and case

If you're unsure about the format, make sure to check out this example and follow the format there.

If this PR isn't a submission, that's okay too! Please remove the above boxes and explain what the pull request is for! If there's anything about this PR template that seems confusing, send a message in #hackpad in slack!

@vercel
Copy link

vercel bot commented Feb 19, 2025

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

Name Status Preview Comments Updated (UTC)
hackpad ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 24, 2025 5:53pm

Copy link
Collaborator

@M0HID M0HID left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Great work on this but can you remove all the unnecessary files please? e.g. the backups and all the examples you have. Thanks!

@spacevini8
Copy link
Member Author

I removed some files, that good?

Copy link
Contributor

@cheyao cheyao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM #4

Copy link
Contributor

@return-misaPuding return-misaPuding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR #634 , LGTM! i´ll let you cook

Copy link
Collaborator

@M0HID M0HID left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Theres still wayy too many files in this pr! Could you delete all the unused files and libraries (you dont need the whole MX_V2 library, just the footprints you've used!). Once you do that I can approve this!

@spacevini8
Copy link
Member Author

I really still don't understand what specifically you want me to delete, but sure, I tried

Copy link
Collaborator

@Taran-the-Idiot Taran-the-Idiot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hihi, can you please delete the assets folder in your pcb folder
Also can you move you gerbers and stls to a separate folder labeled production. you can see it in the guide https://hackpad.hackclub.com/submitting

@vercel
Copy link

vercel bot commented Oct 6, 2025

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

A member of the Team first needs to authorize it.

@spacevini8
Copy link
Member Author

why do I need to delete the assets folder? I've been at this for like, more than a year, but sure, I'll delete the assets folder.

@Taran-the-Idiot
Copy link
Collaborator

check #hackpad-announcements on slack. tldr reviews are finally happening. also no libraries in your repo

@spacevini8
Copy link
Member Author

there, I did what you asked, anything else?

Copy link
Collaborator

@Taran-the-Idiot Taran-the-Idiot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay looks good, approved

@Taran-the-Idiot Taran-the-Idiot merged commit 9fedd28 into hackclub:main Oct 6, 2025
1 check failed
@spacevini8
Copy link
Member Author

yippie!!!

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.

5 participants