Skip to content
This repository has been archived by the owner on Nov 2, 2021. It is now read-only.

Fixes the weird placeholder strings appearing when language was changed #2508

Closed
wants to merge 2 commits into from

Conversation

sbhrwlr
Copy link

@sbhrwlr sbhrwlr commented Jun 30, 2021

Fixes #2507

A brief description of what changes your PR introduces. The better this description, the quicker we can review, exchange feedback, and merge!
After changing language the placeholder was displaying weird strings and not the cities suggestion you added it was happening because of useCallback function that starts running and do not terminate the priveous instance of useCallback I added language dependency to useCallback and function will terminate whenever the language state changes.

Fixes #2507

✔️ Compiles and passes lint tests
✔️ Properly formatted
✔️ Tested on desktop
✔️ Tested on phone

Before Fixing Screenshot :

initially
Initially wrong strings

After Fixing Screenshots

now

nowtwo
:

Copy link

@rahgurung rahgurung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not push package-lock.json.

@sbhrwlr sbhrwlr closed this Jul 4, 2021
@sbhrwlr sbhrwlr deleted the langissue branch July 4, 2021 08:05
@sbhrwlr
Copy link
Author

sbhrwlr commented Jul 4, 2021

created new pr for it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

After changing language and then coming back to english, the search placeholder displays weird strings
2 participants