Skip to content

Issue 48#58

Open
Ayaanshaikh12243 wants to merge 2 commits intoarshchouhan:mainfrom
Ayaanshaikh12243:ISSUE-48
Open

Issue 48#58
Ayaanshaikh12243 wants to merge 2 commits intoarshchouhan:mainfrom
Ayaanshaikh12243:ISSUE-48

Conversation

@Ayaanshaikh12243
Copy link
Copy Markdown
Contributor

image image image image image

Copilot AI review requested due to automatic review settings January 27, 2026 13:08
@vercel
Copy link
Copy Markdown

vercel bot commented Jan 27, 2026

@Ayaanshaikh12243 is attempting to deploy a commit to the Arsh Chauhan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Ayaanshaikh12243
Copy link
Copy Markdown
Contributor Author

@arshchouhan please check and merge

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new public “About Us” page to the frontend and introduces basic rate limiting on key backend endpoints to address Issue 48.

Changes:

  • Introduced AboutUsPage with marketing content, metrics, team section, and CTAs, wired into routing and navigation.
  • Updated LandingNavbar to surface an “About Us” link alongside existing marketing/navigation links.
  • Added express-rate-limit to the backend, configuring a general rate limiter plus stricter limits for auth and upload-related routes.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Frontend/src/pages/AboutUsPage.jsx New About Us page component with hero, story, stats, values, team, vision, and CTA sections, using existing landing layout components.
Frontend/src/components/LandingNavbar.jsx Adds an “About Us” navigation link that routes to the new page, consistent with other navbar links.
Frontend/src/App.jsx Registers the /about-us route and mounts AboutUsPage so it is reachable via routing.
Backend/package.json Declares express-rate-limit as a backend dependency to support request throttling.
Backend/index.js Configures and applies a global rate limiter plus auth- and upload-specific rate limiters, and wires them into existing API routes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Ayaanshaikh12243
Copy link
Copy Markdown
Contributor Author

@arshchouhan please check and merge my pr sir thank you

@arshchouhan
Copy link
Copy Markdown
Owner

Under review, kindly wait

@Ayaanshaikh12243
Copy link
Copy Markdown
Contributor Author

@arshchouhan okay sir thank you i will be creating new issues in an hr please assign me that i will start working thank you

@Ayaanshaikh12243
Copy link
Copy Markdown
Contributor Author

@arshchouhan sir please check and merge

@Ayaanshaikh12243
Copy link
Copy Markdown
Contributor Author

@arshchouhan sir ek week hoobagya please check and merge

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