Self nomination branch - #84
Conversation
|
@BabuBahir is attempting to deploy a commit to the sajeetharan's projects Team on Vercel. A member of the Team first needs to authorize it. |
Port the 'Add me to DevGlobe' nomination feature to the new structure: - api/nominate.js -> app/api/nominate/route.js (Next.js App Router) - api/lib/nominate.js -> lib/nominate.js - src/components/AddMeModal.jsx/.module.css -> components/ - Wire AddMeModal + 'Add Me To Globe' button into app/page.jsx and components/Header.jsx - Keep review-nominations script; drop Vite-era server.js/src/api files
|
I was able to add me via UI and Nominate @sajeetharan |
There was a problem hiding this comment.
Pull request overview
Implements the “Add me to DevGlobe” self-nomination flow from Issue #27 by adding a header CTA + modal form, a Next.js API endpoint to validate/store nominations in Cosmos DB, and an admin CLI script to review and promote nominations into the main developers dataset.
Changes:
- Added “Add Me To Globe” button in the header that opens a modal to submit GitHub username + optional location.
- Implemented
POST /api/nominateand shared nomination logic (lib/nominate.js) to validate GitHub usernames and store pending nominations in Cosmos DB. - Added an admin review CLI (
scripts/review-nominations.js) and wired it intopackage.json.
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| styles/main.css | Adds button variants for the new “Add Me” CTA and primary modal actions. |
| components/Header.jsx | Adds “Add Me To Globe” header button and new onAddMe prop. |
| components/AddMeModal.jsx | Implements the self-nomination modal UI and client-side submit behavior. |
| components/AddMeModal.module.css | Styles for the nomination modal overlay/content. |
| app/page.jsx | Wires modal open/close state into the home page and passes handler to header. |
| app/api/nominate/route.js | Adds API endpoint that delegates to shared nomination logic. |
| lib/nominate.js | Shared server-side nomination validation + Cosmos storage logic. |
| scripts/review-nominations.js | Admin CLI for listing/approving/rejecting nominations; approves into developers container. |
| package.json | Adds review-nominations script entry. |
| .gitignore | Ignores dist/ output directory. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Fixed the changes and tested locally , was able to nominate and approve user @sajeetharan |
|
Thanks for the PR @BabuBahir , looks good! Expecting more contributions and invite other collaborators! |
Resolves #27