-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add search feature #98
Add search feature #98
Conversation
@lucianghinda have a look and on that too https://github.com/ShortRuby/rubyandrails.info/issues/23#issuecomment-1682892929 |
@obaidashfaq20 I am sorry for the delay in reviewing and merging this. I am in a vacation and will manage to review this next week. |
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.
@obaidashfaq20 Thank you for this PR. It looks good to be merged.
I have some improvements that need to be done, but they are related to the general structure of the project, so I don't want to keep this PR based on those.
I will merge this and then do myself a small PR to improve a bit the view of the search.
Work in progress!
Description to be added