-
-
Notifications
You must be signed in to change notification settings - Fork 63
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.
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
Merge branch 'main' of https://github.com/ajaynegi45/Uttarakhand-Culture-NewUI #209
Conversation
@Mallik-vinukonda is attempting to deploy a commit to the ajaynegi45's projects Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 Thank you @Mallik-vinukonda for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your prompt work on this issue. I appreciate your dedication to the project.
I have reviewed your work, and I need some changes.
94:6 Error: 'Card' is not defined. react/jsx-no-undef
103:6 Error: 'Card' is not defined. react/jsx-no-undef
112:6 Error: 'Card' is not defined. react/jsx-no-undef
121:6 Error: 'Card' is not defined. react/jsx-no-undef
130:6 Error: 'Card' is not defined. react/jsx-no-undef
139:6 Error: 'Card' is not defined. react/jsx-no-undef
148:6 Error: 'Card' is not defined. react/jsx-no-undef
157:6 Error: 'Card' is not defined. react/jsx-no-undef
166:6 Error: 'Card' is not defined. react/jsx-no-undef
./src/components/about/About.tsx
21:9 Warning: Using <img>
could result in slower LCP and higher bandwidth. Consider using <Image />
from next/image
to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element @next/next/no-img-element
48:9 Warning: Using <img>
could result in slower LCP and higher bandwidth. Consider using <Image />
from next/image
to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element @next/next/no-img-element
./src/components/ui/TrekCard.tsx
125:9 Warning: Using <img>
could result in slower LCP and higher bandwidth. Consider using <Image />
from next/image
to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element @next/next/no-img-element
info - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
Error: Command "npm run build" exited with 1
Once again, thank you for your contribution! ❤️
@@ -14,11 +14,15 @@ | |||
"@neondatabase/serverless": "^0.9.5", | |||
"@vercel/speed-insights": "^1.0.12", | |||
"bcryptjs": "^2.4.3", | |||
"dotenv": "^16.4.5", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove it
"next-auth": "^5.0.0-beta.22", | ||
"nodemailer": "^6.9.15", | ||
"pg": "^8.13.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove it
i got errors while setting up the project , so i did it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove it
i got errors while setting up the project , so i did it
Error: No database connection string was provided to neon()
. Perhaps an environment variable has not been set?
This error happened while generating the page. Any console logs will be displayed in the terminal window.
Source
src/lib/drizzle.ts (8:19) @ connectionString
6 | const connectionString = process.env.DATABASE_URL!;
7 |
8 | const pool = neon(connectionString)
| ^
9 |
10 | export const db = drizzle(pool, { schema })
Call Stack
"drizzle-kit": "^0.24.2", | ||
"drizzle-orm": "^0.33.0", | ||
"next": "14.2.5", | ||
"framer-motion": "^11.11.10", | ||
"gsap": "^3.12.5", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
framer-motion and gsap use only framer.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
framer-motion and gsap use only framer.
Ok i will
justify-content: center; | ||
align-items: center; | ||
margin: 3rem 0 5rem 0; | ||
color: #ffff; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why you change the spacing. Who said you to change it. Make it as it was earlier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why you change the spacing. Who said you to change it. Make it as it was earlier
ok , that's my bad , i apoloize for that
Screen.Recording.2024-10-26.at.2.57.03.PM.mp4