Skip to content

Conversation

@Mounika-max-348
Copy link

@Mounika-max-348 Mounika-max-348 commented Dec 5, 2025

What kind of change does this PR introduce?
Bugfix — prevents submitting feedback with an empty comment and improves the UI experience.

Issue Number:

Screenshots/videos:
Screenshot 1
Screenshot 2

If relevant, did you update the documentation?
Not applicable.

Summary
This PR:

  • Validates feedback-comment and blocks empty submissions.
  • Shows an inline error: “Feedback comment cannot be empty.”
  • Adds success and error popups for improved UX.
  • Keeps existing layout and logic intact.

Does this PR introduce a breaking change?
No.

Checklist

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

Hi @Mounika-max-348! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview 0de8f76

@codecov
Copy link

codecov bot commented Dec 5, 2025

Codecov Report

❌ Patch coverage is 78.78788% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.94%. Comparing base (2dd1d24) to head (0de8f76).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
components/DocsHelp.tsx 78.78% 7 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##              main    #1970      +/-   ##
===========================================
- Coverage   100.00%   98.94%   -1.06%     
===========================================
  Files           30       30              
  Lines          633      662      +29     
  Branches       196      202       +6     
===========================================
+ Hits           633      655      +22     
- Misses           0        7       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

Hi @Mounika-max-348! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

1 similar comment
@github-actions
Copy link

github-actions bot commented Dec 5, 2025

Hi @Mounika-max-348! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready to review

Development

Successfully merging this pull request may close these issues.

🐛Bug: Feedback Form Submits Empty String

1 participant