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

Fixed ESLint syntax errors across frontend #639

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

irais-valenzuela
Copy link
Member

Fixes #609

Changes

Resolved 33 ESLint syntax errors to improve code quality and maintainabilty. No functional changes, just syntax fixes.

  • Enhanced TypeScript typings for better type safety and clarity.
  • Improved prop validations by integrating PropTypes and TypeScript interfaces.
  • Refactored functions, interfaces, and event handlers to use explicit types, eliminating any.

Notes:

  • Some ESLint rules were selectively disabled with inline comments to accommodate ongoing development and pending design reviews.
  • There was an error due to conflicting Tailwind CSS rules. I temporarily disabled the linter for this issue, but it might be worth checking with the UX team to determine the appropriate styling approach.

Screenshots, if applicable

Title

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.

Resolve all eslint errors
1 participant