Adding for printers text field section while shipping.#203
Open
Shreerang912 wants to merge 8 commits intohackclub:mainfrom
Open
Adding for printers text field section while shipping.#203Shreerang912 wants to merge 8 commits intohackclub:mainfrom
Shreerang912 wants to merge 8 commits intohackclub:mainfrom
Conversation
Refactor project listing page to improve filtering and search functionality. Updated project rendering and added filter chips for better user experience.
Add a new for_printers text column (NOT NULL, default '') to the project table via a Drizzle migration and expose it throughout the app. Updated schema (forPrinters) with a comment, included the field in server load handlers for project pages and the ship flow, and wired the ship form to accept and persist up to 3000 characters. UI: show a collapsible "For Printers" block on the project page and add a textarea on the ship page (with placeholder and visibility note). Files changed: migration, DB schema, project page load + UI, ship page load + UI + actions.
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.
Add "For Printers" field to project shipping
What's changed
Adds an optional "For Printers" text field to the ship page, allowing project authors to leave notes specifically for whoever ends up printing their model.
Why
Printers often need specific technical details that don't belong in the public project description — things like recommended supports, tolerance notes, material suggestions, or tricky parts to watch out for. This gives authors a dedicated place to put that info.
Behaviour
/dashboard/projects/[id]/ship) with a 3000 character limit/dashboard/projects/[id]) — it does not appear on project cardsFiles changed
src/lib/server/db/schema.ts— addsforPrinterscolumn to theprojecttabledrizzle/0036_for_printers_field.sql— migration to add the column to the databasesrc/routes/dashboard/projects/[id]/ship/+page.svelte— adds the textarea to the ship formsrc/routes/dashboard/projects/[id]/ship/+page.server.ts— reads and saves the field on submissionsrc/routes/dashboard/projects/[id]/+page.server.ts— exposesforPrintersto the project detail pagesrc/routes/dashboard/projects/[id]/+page.svelte— renders the collapsible "For Printers" sectionAfter merging
This PR includes a database migration. Before deploying, the
for_printerscolumn needs to be added to the database by running the migration indrizzle/0036_for_printers_field.sql. Without this step the site will crash for anyone trying to ship a project.