Skip to content
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

Implement reusable search component #55

Open
arkid15r opened this issue Nov 3, 2024 · 8 comments
Open

Implement reusable search component #55

arkid15r opened this issue Nov 3, 2024 · 8 comments
Assignees
Labels

Comments

@arkid15r
Copy link
Collaborator

arkid15r commented Nov 3, 2024

Like the approach for a shared component for all items, we want a search bar that works across all pages (Contribute, Projects, Chapters, and Committees).

This search bar should be flexible and configurable, allowing it to connect with the backend and provide the right search features for each page.

@arkid15r arkid15r added this to the Implement Nest frontend milestone Nov 3, 2024
@Yashgupta9330
Copy link
Contributor

I want to work on this issue can you please assign me this issue?

@arkid15r
Copy link
Collaborator Author

arkid15r commented Nov 5, 2024

You got it

@tsu-ki
Copy link
Collaborator

tsu-ki commented Nov 27, 2024

@Yashgupta9330 Are you still working on this issue? I noticed that you've also assigned yourself another issue(#68). So, if you aren't working on this, then I'd like to take over

@Yashgupta9330
Copy link
Contributor

I have made the changes and am waiting for the Django PR to be merged so that I can test it.

@arkid15r
Copy link
Collaborator Author

I have made the changes and am waiting for the Django PR to be merged so that I can test it.

Hey @Yashgupta9330 what PR blocks you (not sure if it's something I can help you with)?

@Yashgupta9330
Copy link
Contributor

Yashgupta9330 commented Nov 27, 2024

@arkid15r To make API calls, we first need to resolve the CORS issue in Django.

@arkid15r
Copy link
Collaborator Author

Gotcha! Yes, I've reviewed #100 and waiting for the comments to be addressed. I hope it'll be merged today.

You can also reference the PRs in your issue replies as a blockers. That'd save some time for us.

@arkid15r
Copy link
Collaborator Author

@Yashgupta9330 just FYI -- the #100 has been merged.

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

No branches or pull requests

3 participants