Skip to content

Conversation

@YadValRio
Copy link

@YadValRio YadValRio commented Oct 14, 2025

Overview

Added documentation to following components:
Github Button
SingleGradesInfo
SingleProfInfo
Split
StickySide
WhatsNew

What Changed

Added a brief description, props table, and brief code example

YadValRio and others added 4 commits September 23, 2025 14:49
TODO: Review and see with project lead if more info is appropriate.
Added README and documentation markdown files for several common components (GitHubButton, SingleProfInfo, Split, StickySide, WhatsNew) and expanded the SingleGradesInfo README with usage examples. Updated ESLint and Prettier dev dependencies to latest versions in package.json, but left out of commit.
I used `npm run format` and `npm run lint` to properly format the documents I edited to pass the project checkers.
@YadValRio YadValRio requested a review from egsch as a code owner October 14, 2025 23:36
@vercel
Copy link

vercel bot commented Oct 14, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
utd-trends Ready Ready Preview Comment Nov 11, 2025 2:39am

@YadValRio YadValRio closed this Oct 14, 2025
@YadValRio YadValRio reopened this Oct 14, 2025
@egsch
Copy link
Contributor

egsch commented Nov 11, 2025

Hey @YadValRio my sincere apologies for the super super late response! These look really good, thank you for your work!

Just FYI, in the code samples we generally try to use the component directly rather than recreating so we have a reference on its use that we won't need to update every time something inside the component changes. I made a couple of updates to the samples based on this; sorry for not letting you know abt that earlier, I understand there were a couple of tricky spots in these components. If you could look over the changes I made and try to update the last code sample -- for SingleGradesInfo-- along the same lines yourself that would be great! Or let me know if you'd rather I finish it up, sorry again for responding so late!

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.

3 participants