Repository navigation
Conversation
Collaborator
Author
|
@narcisonunez Can you please look into this. |
… commits, and add anonymous card
Collaborator
Author
|
My changes are clean. CI failed because |
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.
Description
This PR adds a new
/contributorspage to highlight the top open-source developers supporting Dokploy as a leaderboard. By showcasing their names and accurate contribution counts directly on the website, the goal is to show appreciation, encourage the community to come up with new solutions, and drive overall engagement. The PR also includes a few minor layout and navigation fixes.Main Feature: The New Contributors Page
/stats/contributorsendpoint exclusively, the only endpoint that returns mathematically correct commit totals matching the GitHub Web UI. No inflated or estimated numbers.per_page=1call withLinkheader parsing determines the total number of contributors without fetching them all, used to display a summary card.202 Acceptedcalculation status with a retry loop, and filters out non-human "bot" accounts. If the GitHub API fails or rate-limits, it falls back to a 1-hour stale memory cache.hidden md:flex) to instantly hide the bottom 20 items, resulting in zero React hydration mismatch errors.avatars.githubusercontent.comtonext.config.jsto support remote avatars.Minor Fixes & Navigation Cleanup:
Header.tsx) to make room for the new "Contributors" link. The "Jobs" link remains intact inside the "Resources" dropdown.Footer.tsx), and updated the AI documentation file (llms.txt) to reference "Jobs" instead of "Careers".Note for Reviewers (Architectural Judgements)
/stats/contributors? The paginated/contributorsendpoint returns all 365+ users, but it inflates commit counts for top users. The/stats/contributorsendpoint returns the accurate commit counts (matching the GitHub Web UI) but is capped at ~100 users. Rather than fetching both and merging (a dual-fetch strategy), we chose data accuracy over completeness — the top ~100 with correct numbers, plus a summary card for the rest.api/contributorsroute is set up to match the existing caching patterns used ingithub-starsandgithub-contributors. It uses a 1-hour in-memory cache, combined with a fallback helper that serves stale data and forces a cooldown if GitHub goes down.xl? : The desktop navbar requires approximately1060pxof horizontal space. Changing the collapse breakpoint toxlguarantees the desktop navbar only renders when there is at least1216pxof container width. This completely eliminates the horizontal over-scroll bug while safely displaying the hamburger menu on anything smaller.Checklist
llms.txt) is updated to reflect these changespnpm run website:devand changes were verifiedbiome checkScreenshots
1. The New Contributors Page:
2. Navbar Layout Fix (less screen width):