Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Form for testing #14

Merged
merged 39 commits into from
Sep 29, 2024
Merged

Form for testing #14

merged 39 commits into from
Sep 29, 2024

Conversation

joywin2003
Copy link
Contributor

No description provided.

@joywin2003
Feat: Added the start registration-form
5b3272b
@joywin2003
Feat: Add confirmation dialog component
8f9c83e
@joywin2003
Changed the form to an accordian and added additional styling
30493f3
@joywin2003
Refactor registration form styling and layout for the accordian and d…
a49b314 
…elete button
@joywin2003
Refactor: Added borders when the user open the accordion
98fff37
@joywin2003
Refactor: Rearranged the fields and prevent add registration when the…
d20d454 
… previous form is not verified
@joywin2003
Feat: Added sonner toasts
49a8ed6
@joywin2003
Refactor: Update registration form to disable certain fields based on…
267ebf4 
… designation
@joywin2003
Refactor: Added type to registration and fixed the details entry
074e3ed
@joywin2003
Refactor: Update registration form to add verification button
347e915
@joywin2003
Feat: register only single user form
43551a0
@joywin2003
Feat: Added upload thing to upload images
6efd172
@joywin2003
added auth() to get user from session
b183c23
@joywin2003
Refactor: Update auth function to get user from session
8958976
@joywin2003
fix: remove auth() function in utils which was causing the "url undef…
8917b3f 
…ined aka env undefined"
@joywin2003
Added ssr plugin for Improving SSR for upload thing
22091a2
@joywin2003
Add styles for upload thing in registration form
9534192
@joywin2003
added custom uploadthing function using hook
78e24ac
@joywin2003
added loading and completed animation
325def3
@joywin2003
Feat: uploading collegeid and photo when i click on register
8fc8786
@joywin2003
Refactor: Improve registration form and upload functionality
31fe434
@joywin2003
Refactor: Add @hookform/resolvers and react-hook-form package for imp…
3f84fac 
…roved form functionality
@joywin2003
Refactor: code optimisation, using copilot
646462c
@joywin2003
Feat: Add payment component to registration page
638245e
@joywin2003
Refactor: made getprice to return initial, discount and final price
19d9811
@joywin2003
Refactor: formatted the few files on roles and register
bd2f063
@joywin2003 joywin2003 added the enhancement New feature or request label Sep 26, 2024
Vyshnav001
Vyshnav001 requested changes Sep 26, 2024
View reviewed changes
src/app/layout.tsx Outdated Show resolved Hide resolved
src/components/payment.tsx Outdated Show resolved Hide resolved
@joywin2003
Refactor: requested changes from vyshnav
8ba8eb0 
- moved constants to constant/index
- shifted layout wrappers to provider
- used constants
@joywin2003
Copy link
Contributor Author

Added the requested changes

@joywin2003
Copy link
Contributor Author

Error: to fetch users: q [Error]: Dynamic server usage: Route /api/users couldn't be rendered statically because it used `request.url`. See more info here: https://nextjs.org/docs/messages/dynamic-server-error at W (/home/runner/work/website/website/node_modules/next/dist/compiled/next-server/app-route.runtime.prod.js:6:21106) Generating static pages (6/13) at Object.get (/home/runner/work/website/website/node_modules/next/dist/compiled/next-server/app-route.runtime.prod.js:6:28459) at p (/home/runner/work/website/website/.next/server/app/api/users/route.js:1:649) at /home/runner/work/website/website/node_modules/next/dist/compiled/next-server/app-route.runtime.prod.js:6:36264 at /home/runner/work/website/website/node_modules/next/dist/server/lib/trace/tracer.js:140:36 at NoopContextManager.with (/home/runner/work/website/website/node_modules/next/dist/compiled/@opentelemetry/api/index.js:1:7062) at ContextAPI.with (/home/runner/work/website/website/node_modules/next/dist/compiled/@opentelemetry/api/index.js:1:518) at NoopTracer.startActiveSpan (/home/runner/work/website/website/node_modules/next/dist/compiled/@opentelemetry/api/index.js:1:18093) at ProxyTracer.startActiveSpan (/home/runner/work/website/website/node_modules/next/dist/compiled/@opentelemetry/api/index.js:1:18854) at /home/runner/work/website/website/node_modules/next/dist/server/lib/trace/tracer.js:122:103 { description: "Route /api/users couldn't be rendered statically because it used `request.url`. See more info here: https://nextjs.org/docs/messages/dynamic-server-error", digest: 'DYNAMIC_SERVER_USAGE' }

Getting this error from your admin check in middlware.ts

@Vyshnav001
Copy link
Collaborator

Error: to fetch users: q [Error]: Dynamic server usage: Ro

which middleware is that ?
i don't remember creating a middleware

Vyshnav001
Vyshnav001 approved these changes Sep 27, 2024
View reviewed changes
Copy link
Collaborator

@Vyshnav001 Vyshnav001 left a comment

Choose a reason for hiding this comment

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

Rest everything looks good for me

@joywin2003
Copy link
Contributor Author

then its better to merge it, furthure changes will be done in another PR

@joywin2003 joywin2003 merged commit a80977d into main Sep 29, 2024
2 checks passed
@joywin2003 joywin2003 deleted the form branch October 5, 2024 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vyshnav001 Vyshnav001 Vyshnav001 approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joywin2003 @Vyshnav001