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.

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

Closed
wants to merge 2 commits into from

Conversation

Mallik-vinukonda
Copy link

@Mallik-vinukonda Mallik-vinukonda commented Oct 26, 2024

Screen.Recording.2024-10-26.at.2.57.03.PM.mp4

Copy link

vercel bot commented Oct 26, 2024

@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.

Copy link

@github-actions github-actions bot left a 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.

Copy link

vercel bot commented Oct 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
uttarakhand-culture ❌ Failed (Inspect) Oct 27, 2024 3:33am

Copy link
Owner

@ajaynegi45 ajaynegi45 left a comment

Choose a reason for hiding this comment

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

Hi @Mallik-vinukonda,

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",
Copy link
Owner

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",
Copy link
Owner

Choose a reason for hiding this comment

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

remove it

Copy link
Author

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

Copy link
Author

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",
Copy link
Owner

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.

Copy link
Author

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;
Copy link
Owner

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

Copy link
Author

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

@ajaynegi45 ajaynegi45 closed this Oct 27, 2024
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.

2 participants