Skip to content

fix: trigger stock search on Enter - #233

Open
Ambu-2005 wants to merge 1 commit into
SrigadaAkshayKumar:mainfrom
Ambu-2005:fix/search-enter-key
Open

Ambu-2005 wants to merge 1 commit into
SrigadaAkshayKumar:mainfrom
Ambu-2005:fix/search-enter-key

Conversation

@Ambu-2005

Copy link
Copy Markdown

Issue Title

[BUG] Search functionality doesn't trigger on Enter key

Contributor Details

Closes

Closes #216

Describe Your Changes

  • Added Enter key handling to the stock search input.
  • Pressing Enter now triggers the existing handleSearch function.
  • Existing Search button functionality remains unchanged.

Screenshots

Tested the search functionality using TCS.NS.
Pressing Enter successfully navigated to the TCS.NS stock analysis page.

what are the files you have changed

  • src/components/StockList.jsx

How Has This Been Tested?

  • Started the application locally using npm start.
  • Entered TCS.NS in the search input.
  • Pressed Enter.
  • Verified that the application navigated to the TCS.NS stock analysis page.
  • Verified that the existing Search button functionality remains available.

Type of Change

  • Bug fix (fixes an existing issue)

Checklist

  • My code follows the project’s guidelines and best practices.
  • I have performed a self-review of my code.
  • I have added meaningful comments where necessary.
  • I have updated documentation wherever applicable.
  • My changes do not generate any new warnings or errors.
  • I have included screenshots, graphs, or test logs (if applicable).
  • Any dependent changes have been merged and documented.

@github-actions

Copy link
Copy Markdown

Thanks for opening this Pull Request! A maintainer will review it shortly. Please make sure you've followed all contribution guidelines.

Join our Discord Community for discussions, questions, and real-time collaboration!

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.

[BUG] Search functionality doesn't trigger on Enter key

1 participant