Conversation
|
@copilot clerk sign in and sign up urls set in the component props instead of env variables. remove from .env.example the ones that we don't need. |
Co-authored-by: danielpaul <333233+danielpaul@users.noreply.github.com>
Co-authored-by: danielpaul <333233+danielpaul@users.noreply.github.com>
I've updated the implementation to remove the unused Clerk environment variables from
Only kept Changes in commit 2b867c6. |
| end | ||
| end | ||
|
|
||
| def create_signup |
There was a problem hiding this comment.
we don't need create_signup. this does not do anything.
There was a problem hiding this comment.
Removed the create_signup method as it was unnecessary. Clerk handles authentication entirely on the frontend through the __session cookie, so POST endpoints for token validation aren't part of the standard flow. Changes in commit 012a1d2.
| end | ||
| end | ||
|
|
||
| def create |
There was a problem hiding this comment.
Removed the create method as it was unnecessary. The POST /sign_in endpoint isn't needed since Clerk manages authentication through its own frontend flow. Changes in commit 012a1d2.
|
@copilot work on the changes |
Co-authored-by: danielpaul <333233+danielpaul@users.noreply.github.com>
I've addressed the feedback by removing the unnecessary Warning Firewall rules blocked me from connecting to one or more addressesI tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
This PR implements a complete sign-up page that mirrors the existing sign-in functionality, providing users with a consistent authentication experience.
What's Added
New Sign-Up Page: Created
app/frontend/pages/sessions/sign_up.tsxusing Clerk's<SignUp>component, following the same pattern as the existing sign-in page for consistency.Backend Routes & Controller: Added
GET /sign_uproute andsign_upmethod inSessionsControllerto handle the sign-up page rendering.Complete Test Coverage: Added comprehensive tests for the sign-up GET endpoint and existing Clerk authentication flow, ensuring proper authentication behavior.
Environment Variables Cleanup: Removed unused Clerk environment variables from
.env.examplesince sign-in and sign-up URLs are set directly in component props instead of environment variables.Key Features
AuthLayoutand styling as the sign-in pageAuthentication Flow
The implementation follows Clerk's standard authentication pattern where:
__sessioncookie that Clerk managesFiles Changed
config/routes.rb- Added sign-up GET routeapp/controllers/sessions_controller.rb- Added sign-up handling methodapp/frontend/pages/sessions/sign_up.tsx- New sign-up page componentspec/requests/sessions_spec.rb- Added comprehensive test coverage.env.example- Removed unused Clerk environment variablesThe implementation maintains the existing authentication patterns while providing the requested sign-up functionality. All tests pass and the code follows the project's linting and formatting standards.
Fixes #26.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.