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 businesses domain #2

Closed
7 tasks done
LauraBeatris opened this issue Dec 2, 2020 · 0 comments · Fixed by #5
Closed
7 tasks done

Add businesses domain #2

LauraBeatris opened this issue Dec 2, 2020 · 0 comments · Fixed by #5

Comments

@LauraBeatris
Copy link
Owner

LauraBeatris commented Dec 2, 2020

Describe the feature request

Add the businesses domain in order to retrieve data about the businesses.

Describe the solution you'd like

The frontend should receive the locations data according to these three filters:
image

A new service should be created for each filter since the logic to parse the data is different.

  • GetOldestBusinessService
  • GetBusinessWithMostLocations

The controllers can be the following:

  • BusinessesController
    • show method receives two types of query params (oldest and most-locations)
    • index method returns all the locations fetched from the data source.

Tasks

  • Create GetOldestBusinessService and add tests
  • Create GetBusinessWithMostLocations and add tests
  • Create GetAllBusinessesService and add tests
  • Create BusinessesController
  • Create BusinessWithMostLocationsController
  • Create OldestBusinessController
  • Create businesses routes

Notes

Return only the first page of businesses

@LauraBeatris LauraBeatris created this issue from a note in Backend Features (To do) Dec 2, 2020
@LauraBeatris LauraBeatris changed the title Add businesses domain: Add businesses domain Dec 2, 2020
@LauraBeatris LauraBeatris linked a pull request Dec 2, 2020 that will close this issue
Backend Features automation moved this from To do to Done Dec 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

1 participant