Skip to content

Conversation

@techieadi4703
Copy link
Contributor

@techieadi4703 techieadi4703 commented Oct 16, 2025

Fixes #16

Summary

Added a debounced search feature for the city input field to reduce unnecessary API calls while typing.
This ensures smoother performance and prevents rate-limit issues when users type quickly.

Changes

  • Implemented debounce logic in the city search component.
  • Updated TODOs list to include the new optimization issue.
  • Verified API caching still works as expected with the debounce delay.

Impact

Improves performance and reduces redundant API hits during user input.

Screen-recording:
@

Screen.Recording.2025-10-16.at.20.23.11.mov

@vercel
Copy link

vercel bot commented Oct 16, 2025

@techieadi4703 is attempting to deploy a commit to the venisha-kalola's projects Team on Vercel.

A member of the Team first needs to authorize it.

@techieadi4703
Copy link
Contributor Author

I have resolved this issue and raised a PR for the same.
Kindly review it and merge if everything looks fine.

@venisha-kalola
Copy link
Contributor

add screenshots of implementation

@techieadi4703
Copy link
Contributor Author

@venisha-kalola I've attached screen-recording for the same.
Kindly check it.

@venisha-kalola
Copy link
Contributor

there are some conflicts showing as merge conflict, please check the latest file n implement changes accordingly so merge conflicts do not occur

@techieadi4703
Copy link
Contributor Author

now check it, I have resolved them.

@venisha-kalola venisha-kalola merged commit 522d89a into commitra:main Oct 17, 2025
1 of 2 checks passed
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.

Weather.jsx: Input debounced search

2 participants