streamline layout and enhance community social proof - #226
Merged
Merged
Conversation
10 tasks
Emmanuel-Tyty
self-requested a review
May 6, 2026 07:16
HarunJr
self-requested a review
May 12, 2026 13:14
HarunJr
previously requested changes
May 12, 2026
Contributor
There was a problem hiding this comment.
Thanks for putting this together. The visual design and the use of live data look fantastic!
I have a few architectural notes to help us polish this up before merging:
- Lockfiles (
website/bun.lock&website/yarn.lock) It looks like bun was used locally. This project strictly uses yarn, so committing bun.lock could cause confusion for other contributors and break our CI.
- Suggestion: Please delete
bun.lock, runyarn installto cleanly restoreyarn.lock, and update the PR.
I have highlighted some Rate limit issues and some nick picks before. other than that I have a few nit picks on the UI design.
- For consistency and visual pull poposes purposes, kindly restore the rounded joint intersect to the original orange rounded corner and center it since you have removed the other
Visit GitHub Repositoryfrom this location.
The goal is for this to be the primary button visitors want to click on, so it should be a different colot i.e: Orange for Prominence.
- As the Scondary button, again, we can maintain the rounded corners but consider having this button lead to the issues page like your initial PR. Seeing as it is a secondary button, consider having it white. This should direct the visitor's eyes after the join intersect button.
fetch-github-data.js is a script that runs automatically at build time. It queries the GitHub API once during compilation and writes a static JSON snapshot to githubData.json. It supports `process.env.GITHUB_TOKEN` for higher rate limits in CI/CD pipelines and local development.
… restyle the pulse section button
…itySection UI styles
… update pulse section UI styling
…ta and update community section UI styles
…imize GitHub data fetching to use local JSON source
…nd update UI styles
…local JSON data, and add TypeScript typing to Root component
…ySection styles for centered layout and updated button branding
… branding and styling
…lation matrix for stale-issue automation
Member
|
Hi @IanoNjuguna, can you fix this conflict, so we can merge this PR ? Thanks |
IanoNjuguna
force-pushed
the
devex-stats_ian
branch
from
September 14, 2026 12:20
2872d7f to
17db1e7
Compare
IanoNjuguna
dismissed
HarunJr’s stale review
September 14, 2026 12:53
Code review threshold is met :)
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.
The purpose of this PR is to make the Community Section more engaging.
Related to #218Type of Change
Check the type of change this PR introduces:
Changes Made
Contributors.tsxcomponent that dynamically fetches and displays real-time contributor data from the GitHub API.Checklist
Ensure you have completed the following steps before submitting your PR:
mainormasterbranch.