feat(AddSnippet): Refactor component for better UX and maintainability

notsoocool / codecache

CodeCache is a dynamic web application designed to streamline the way developers manage and access code snippets. The platform provides a comprehensive library of code snippets, allowing users to save, organize, and retrieve useful code snippets for various programming languages and use cases.

https://codecache.tech

MIT License

18 stars 56 forks source link

feat(AddSnippet): Refactor component for better UX and maintainability #135

Closed yuvraj1107thapa closed 2 weeks ago

yuvraj1107thapa commented 1 month ago

This PR refactors the AddSnippet component to improve user experience and code maintainability.

Changes:

Consolidate form state management
Implement reusable FormField component
Enhance form validation and error handling
Add loading state to prevent multiple submissions

Impact:

Improved user experience with better feedback
Enhanced code maintainability and readability
Reduced complexity and code duplication

Testing:

Verified all form fields work with new state management
Tested form submission with valid and invalid inputs
Ensured error messages display correctly
Checked loading state prevents multiple submissions

No breaking changes.

vercel[bot] commented 1 month ago

@yuvraj1107thapa is attempting to deploy a commit to the notsoocool's projects Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 1 month ago

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

Name	Status	Preview	Comments	Updated (UTC)
codecache	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 26, 2024 0:06am

notsoocool commented 1 month ago

Hi @yuvraj1107thapa,

I appreciate your contribution and the effort you put into this PR. However, before raising a PR, we have certain rules and a code of conduct that need to be followed. You should first be assigned to an issue, then proceed to raise a PR.

Kindly create an issue describing the changes, and once that's done, I will review the code and assign the appropriate labels.

Thanks for understanding!

notsoocool commented 1 month ago

@yuvraj1107thapa

notsoocool commented 1 month ago

@yuvraj1107thapa still its getting failed why deploying, kindly fix it

notsoocool commented 1 month ago

@yuvraj1107thapa if you didn't fix it by today itself, I will close your PR

yuvraj1107thapa commented 1 month ago

@notsoocool Fixed all deployment issues and improved component stability. The code has been tested locally and is working as expected. It is ready for review.

notsoocool commented 4 weeks ago

Sorry for the delayed review, @yuvraj1107thapa. I was about to merge your PR, but I noticed an issue: when clicking "Add Snippets," it only gives a toast notification for "usage cannot be empty," even if all fields are empty. Ideally, it should validate each field in order, starting with the title and then moving to the others. Could you please address this?

notsoocool commented 3 weeks ago

@yuvraj1107thapa any updates??