Skip to content

Fix: Comprehensive Fixes and Enhancements for User Feedback System - #205

Merged
Annanyatiwary4 merged 2 commits into
DevSyncx:mainfrom
kmanish527:bug/feedback
Oct 29, 2025
Merged

Fix: Comprehensive Fixes and Enhancements for User Feedback System#205
Annanyatiwary4 merged 2 commits into
DevSyncx:mainfrom
kmanish527:bug/feedback

Conversation

@kmanish527

Copy link
Copy Markdown
Contributor

🚀 Pull Request Checklist

  • ✅ My issue is assigned to me, and I have not taken up another task simultaneously.
  • 🔁 I have pulled the latest changes from the main branch.
  • 🧪 My code is tested and does not break existing functionality.
  • 📚 I have added/updated documentation wherever necessary.
  • 🧹 My code follows the project’s coding standards.
  • ✍️ My commits are clear and meaningful.
  • 🧾 I have linked the issue this PR addresses with Closes #204 .

📌 Related Issue

Closes #204

🧠 Description

  • A critical backend bug was resolved where a user's userId was being saved even on anonymous submissions. This was fixed by making the userId optional in the database model and updating the controller to conditionally add the ID only if the submission is not anonymous.
image
  • A frontend bug was fixed where the feedback popup would display stale data from the previous submission. The component's state is now programmatically reset every time the modal is opened, ensuring a clean, blank form for the user.
image
  • All native browser alert()s were replaced with modern, non-blocking react-toastify notifications. This logic was centralized within the Feedback to handle success and error states, providing a more professional and seamless user experience.
image

✅ Type of Change

  • 🐞 Bug fix
  • 💡 Feature
  • 🧹 Code cleanup/refactor
  • 🧪 Test cases
  • 📚 Docs update

@vercel

vercel Bot commented Oct 29, 2025

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the Annanya's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

🙌 Thanks @kmanish527 for raising this PR!
Our team will review it soon ⏳ — meanwhile, please make sure your PR follows our contributing guidelines 👉 Contributing Guidelines 🎉

@Annanyatiwary4
Annanyatiwary4 merged commit 465bb29 into DevSyncx:main Oct 29, 2025
1 check failed
@github-actions

Copy link
Copy Markdown

🎉🎉 Thank you for your contribution! Your PR #205 has been merged! 🎉🎉

@Annanyatiwary4 Annanyatiwary4 added enhancement New feature or request Frontend backend hacktoberfest-accepted only for hacktoberfest 2025 candidate labels Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend enhancement New feature or request Frontend hacktoberfest-accepted only for hacktoberfest 2025 candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: userId is saved to the database even when feedback is submitted anonymously

2 participants