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

Add keyword search #11

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

hillsandales
Copy link

I've tried to add some code for enabling the keyword search from the Mouser API. In order to return more than one result (as the part number search does) I changed the return type to a List of parts (each part is a Dictionary) instead of just returning one Dictionary.

@eeintech
Copy link
Contributor

@hillsandales Do you mind fixing the code style? Have you ran run_tests.py and does it pass? Can you add a test for keyword search? Thanks!

@eeintech
Copy link
Contributor

eeintech commented Jul 6, 2024

@hillsandales Please fix the issues and I can review :) Thanks!

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.

None yet

2 participants