Skip to content

Conversation

@LevelSilence
Copy link
Contributor

@LevelSilence LevelSilence commented Dec 29, 2025

Issue: #12


📌 Description

Briefly explain what you implemented in this PR.

  • What problem does it solve?
  • What part of the project does it affect?

This PR sets up authentication on the frontend using Clerk with Next.js App Router. It ensures users can sign in and sign up, and protects private routes like the dashboard using middleware (no UI level checks)


🧩 Issue Reference

Related Issue: #12

Note: Do NOT use words like Closes, Fixes, or Resolves.
Issues are Open for All and should not be auto-closed.


🛠️ What Did You Implement?

Please list the major things you worked on in this PR:

  • Feature implementation
  • UI changes
  • API / backend logic
  • Refactor / cleanup
  • Documentation update
  • Demo / setup task

(Select all that apply)


📁 Workspace Confirmation (MANDATORY)

Since all issues are Open for All, confirm the following:

  • All changes are inside
    contributors/<your-github-username>/
  • Base client/ and/or server/ files were copied from the main directory
  • No files outside my personal workspace were modified

🧪 Testing Performed

Please check what applies:

  • Frontend runs locally without errors
  • Backend server runs locally without errors
  • APIs tested with valid inputs
  • APIs tested with invalid / edge-case inputs
  • No console errors or warnings

📷 Screenshots / Demo (REQUIRED for most PRs)

verified locally


✅ Final PR Checklist

Before submitting, ensure:

  • This PR addresses only one issue
  • Issue number is mentioned at the top
  • Code is readable and well-structured
  • No unnecessary files are included
  • No sensitive data or .env files committed
  • Commit messages follow project conventions
  • PR title is clear and descriptive

📎 Additional Notes for Reviewers

Authentication is enforced using Clerk middleware. UI components do not contain any auth logic


@OpenGitBot
Copy link

Hey @LevelSilence

Thanks for opening this PR 🚀. Mentor will review your pull request soon and till then, keep contributing and stay calm.

Thanks for contributing in OpenCode'25 ✨✨!

@Aryan-coder06
Copy link
Collaborator

Aryan-coder06 commented Dec 29, 2025

This doesn’t meet Issue #12 yet: Middleware must be at the project root as middleware.ts, not under app/middleware/middleware.js.\n- Add /sign-in and /sign-up routes. Ensure ClerkProvider wraps the app. Please update and we’ll re-review.

@OpenGitBot
Copy link

Hey @LevelSilence

You can't make a PR on closed issue. Try out our open issues in OpenCode'25!

@LevelSilence
Copy link
Contributor Author

I’ve updated the implementation by moving the middleware to the project root as middleware.ts, added the sign-in and sign-up routes, and ensured the app is wrapped with ClerkProvider (app/layout.tsx)....
The auth flow and protected routes have been verified locally

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