Updates Tech lead on Shelter_Volunteers and Where is Religion and Material Derailleur Write-up post - #287
Open
MATHEW-SHERENI wants to merge 9 commits into
Open
Updates Tech lead on Shelter_Volunteers and Where is Religion and Material Derailleur Write-up post#287MATHEW-SHERENI wants to merge 9 commits into
MATHEW-SHERENI wants to merge 9 commits into
Conversation
kungfuchicken
requested changes
Aug 31, 2026
kungfuchicken
left a comment
Collaborator
There was a problem hiding this comment.
Thanks @MATHEW-SHERENI , and sorry this sat so long. Good writing here, but I want to redirect it.
- The write-up is a product overview, not a blog post. It describes what MaterialDerailleur is and does, which is useful, but nothing in it reports anything that actually happened while building it.
- "Implementation Insights" says designing around real workflows matters and onboarding matters. Both true of every project, so a reader comes away knowing nothing specific about this one.
- Same with the architecture section. It names React, Express, Prisma, and Postgres without saying why we are using any of them.
Recommendations: - <ove the good descriptive sections into
docs/projects/md/about.md. That page is thin and this would genuinely improve it. - Bring the diagrams with you.
architecture-overview.pnganddonatedItemDetails.pngare committed here but never referenced, and both belong on the product page. - If you want an actual post, pick one thing that happened. The item status model is the interesting one, since you mention items get damaged, repaired, moved, mislabeled, duplicated, and reassigned. Barcode/label printing against real printers would work too, or anything BWorks made you change. A story should be about an event or an accomplishment. If you do a site tour or get registered as a DPG, that's newsworthy. As for writing, target Half this length and talk to me before you draft to work through story ideas.
On the tech lead updates, those are fine and still accurate for fall. Pull them into their own PR and I'll merge it this week so you're caught up with everyone else. Two fixes first:
- Drop the
prior Tech Leadfield. No other page has it. Move Supraja and Jacob into the Developers list as(alumni, prior tech lead). - Nothing in
src/data/people/got updated. Supraja and Jacob need to move toprevTechLeads.json, and you need to be incurrentTechLeads.json.
Cleanup, wherever the content ends up:
.idea/.gitignoreshouldn't be committed. Drop it and add.idea/to.gitignore.- The blog folder name has a U+2011 non-breaking hyphen in it instead of a regular one, between "Material" and "Derailleur". Invisible in most editors, breaks shell globbing. Rename to plain ASCII. Your slug is explicit so the URL doesn't change.
- The captions on
donatedItem.pnganddonatedItems.pnglook swapped. - The official spelling is MaterialDerailleur, one word. The post uses "Material-Derailleur" throughout, and
about.mdalready uses the canonical form. - "more time-serving their communities" should be "more time serving".
- The
date:line you added to the Mithridatium post is a fair fix but unrelated, and I have other work in flight on that post. Pull it out.
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.
This pull request adds the new Material‑Derailleur article to the OSS website, including a full project overview, architecture breakdown, and workflow. and Updates Tech lead on Shelter_Volunteers and Where is Religion