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

added search_by feature for breed search service #7

Merged
merged 5 commits into from
Nov 23, 2020

Conversation

mapra99
Copy link
Owner

@mapra99 mapra99 commented Nov 23, 2020

The /breeds/top-searches endpoint depends on BreedSearchService to look up info for a specific breed. This service was currently searching by breed name, not breed id. Added this functionality to be used from the TopBreedsService :)

@mapra99 mapra99 temporarily deployed to cats-wiki-top-searches--54hwpk November 23, 2020 01:04 Inactive
@mapra99 mapra99 temporarily deployed to cats-wiki-top-searches--54hwpk November 23, 2020 01:09 Inactive
@mapra99 mapra99 temporarily deployed to cats-wiki-top-searches--54hwpk November 23, 2020 01:17 Inactive
@mapra99 mapra99 temporarily deployed to cats-wiki-top-searches--54hwpk November 23, 2020 01:21 Inactive
@mapra99 mapra99 temporarily deployed to cats-wiki-top-searches--54hwpk November 23, 2020 02:23 Inactive
@mapra99 mapra99 merged commit a9417c4 into development Nov 23, 2020
@mapra99 mapra99 deleted the top-searches-endpoint-fix branch December 27, 2020 18:28
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.

1 participant