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

Sma 21 add endpoint that returns all sports #14

Merged
merged 13 commits into from
Feb 3, 2024

Conversation

kz44
Copy link
Collaborator

@kz44 kz44 commented Jan 27, 2024

Task: Add endpoint that returns all sports

Description:

  • Make a new controller SportsController
  • Add new Service called SportsService
  • the path to the request should be /api/v1/sports/all

MatejFrnka
MatejFrnka previously approved these changes Jan 27, 2024
* @return paginated list of SportDTO
*/
@GetMapping("/all")
public List<SportDTO> getSports(final Pageable pageable) {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I realised that we might not need pagination for sports because there will probably only be at most 50 different sports. But lets keep it for now as an example

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I miss understood it sorry. I thought later there will be someting filter for it too like search in it or something like that

ivamach
ivamach previously approved these changes Jan 27, 2024
Johna91
Johna91 previously approved these changes Jan 27, 2024
…nto SMA-21_Add_endpoint_that_returns_all_sports

# Conflicts:
#	backend/sportsmatch/src/main/java/com/sportsmatch/repositories/SportRepository.java
@kz44 kz44 dismissed stale reviews from Johna91, ivamach, and MatejFrnka via 651f561 January 30, 2024 17:14
markpernia
markpernia previously approved these changes Feb 1, 2024
@MatejFrnka MatejFrnka merged commit 210a1f7 into develop Feb 3, 2024
6 checks passed
@MatejFrnka MatejFrnka deleted the SMA-21_Add_endpoint_that_returns_all_sports branch February 3, 2024 08:10
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.

5 participants