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

Single Race Lookup #4

Open
wants to merge 11 commits into
base: feat-status_field
Choose a base branch
from
Open

Conversation

ashleyjlive
Copy link
Owner

Description###
Allow for a single race lookup by ID.

Purpose

In some cases, it is not necessary to use a bulk lookup like list-races, instead, only needing to fetch a single ID. This P/R implements this behaviour.

racing.proto:
• Defined GetRace request which accepts the identifier of the race.
queries.go:
• Defined query for `getRace` requests.
races.go:
• Implemented `Get` function which returns a single race and optionally an error.
racing.go:
• Added racing interface and implementation for accessing racesRepo.
• Describe `id` parameter.
races.go:
• Describe return value for Get function.
• Provide comment for GetRaceRequest use case.
• Added test case for `Get` request.
races.go:
• Fixed compilation error due to bad merge.
• Document introduction of `GetRace` rpc call.
• Fixed invalid testType of "db" for races_test.
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