-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
Searchfile added #577
Searchfile added #577
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi there, Thanks a lot for your first pull request, we'll be reviewing it asap :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
requested changes
are you working on this? |
Yes, I will create PR till tomorrow morning please give me time. |
Oh it's alright, just wanted to confirm! :) |
|
Okay sure :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
today I'll check the alternatives as to using something like full text search.You may try researching as well, please
Ok |
Hi @BhartiNagpure check these stack overflow answers: Read them, and implement the free solution |
Thanks alot |
@narayan954 you can check now I am fetching data after writing searchtext from the firebase. |
alright! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
review all changes
yeah sure, on it 🫡 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me!
Thanks for your contribution! :) |
Its my pleasure 🤝 |
Search bar added
close #510