Skip to content

fix: remove unused resetFormFields dead code in submit handler#94

Merged
jagdish-15 merged 2 commits into
codepvg:mainfrom
akshara200829-lgtm:fix/remove-resetFormFields-dead-code
Jun 3, 2026
Merged

fix: remove unused resetFormFields dead code in submit handler#94
jagdish-15 merged 2 commits into
codepvg:mainfrom
akshara200829-lgtm:fix/remove-resetFormFields-dead-code

Conversation

@akshara200829-lgtm

Copy link
Copy Markdown
Contributor

Description

Removes the unused resetFormFields() function that was defined inside the submit event listener but never called anywhere in the file.

Linked Issue

Fixes #70

Changes Made

Deleted the dead resetFormFields() function (~5 lines) from inside the submit event handler in frontend/registration.html. No logic was modified — only the unused function was removed.

Type of Change

  • Bug fix
  • New feature
  • UI/Visual update
  • Documentation update
  • Refactor

Testing

  • Tested locally
  • Tested on mobile viewport (if applicable)
  • No console errors introduced

Checklist

  • My code follows the project's coding style
  • I have formatted my code locally using Prettier
  • I have performed a self-review of my code
  • My changes generate no new warnings or errors
  • I have updated documentation if required
  • I have linked the relevant issue

Screenshots / Screen Recording

N/A

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Thank you for submitting a pull request.

Please ensure your changes comply with the project's contribution guidelines and that all workflow checks pass successfully.

A maintainer will review your submission shortly.

jagdish-15
jagdish-15 previously approved these changes Jun 3, 2026
@jagdish-15

Copy link
Copy Markdown
Collaborator

/format

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

I have successfully run Prettier and pushed the formatting fixes to this PR.

Note for Contributors: Because this commit was pushed by a bot, GitHub will not automatically re-run the CI checks. To trigger them to pass, you must either:

  • Push an empty commit locally (git commit --allow-empty -m "Trigger builds" and push)
  • Close and immediately reopen this Pull Request.

@jagdish-15 jagdish-15 merged commit 9ea6bfc into codepvg:main Jun 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] resetFormFields() is dead code — defined but never called in registration submit handler

2 participants