feat: add sponsor badge UI integration#1341
Conversation
|
@srinidhi-2006-bit is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
Implemented the sponsor badge UI integration for leaderboard and public profile pages. Verified locally that:
The PR includes:
Could you please review the PR and add the Thank you! |
|
Implemented the sponsor badge UI integration for leaderboard and public profile pages. Verified locally that:
The PR includes:
Could you please review the PR and add the Thank you! |
|
please review this and merge the PR |
|
Thanks @srinidhi-2006-bit for the sponsor badge feature! The badge component and profile integration look good. However, the PR bundles a Jest setup ( Please:
The sponsor badge feature itself is valuable — just needs the test framework cleaned up! |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Hi @srinidhi-2006-bit! The sponsor badge UI is a nice feature! However, this PR adds Jest (jest.config.js, jest.setup.js, @testing-library/jest-dom, etc.) but the project uses Vitest (see vitest.config.ts). Please:
- Remove the Jest config files (
jest.config.js,jest.setup.js) and Jest dev dependencies frompackage.json - Rewrite the test in
test/using Vitest (import { describe, it, expect } from 'vitest') - Also note: there's no DB migration to add the
is_sponsorcolumn — the vercel.json cron should also be changed to daily (0 0 * * *) if the sync runs via cron
Thanks!
Summary
Added a reusable SponsorBadge component and integrated sponsor badge support into leaderboard and public profile UI.
Closes #1055
Type of Change
Changes Made
SponsorBadgecomponentPublicProfileDatatype with sponsor supportHow to Test
Steps for the reviewer to verify this works:
Run:
Open:
Verify sponsor badge appears beside sponsor usernames
Open:
Verify sponsor badge appears beside profile username
Run:
Verify there are no lint or TypeScript errors
Screenshots (if UI change)
Added sponsor badge UI beside usernames in leaderboard and public profile pages.
Checklist
npm run lintpasses locallynpm run type-check)