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

Improved search software API endpoint #211

Conversation

JonathanLeeWH
Copy link
Contributor

Search API endpoint

  • Added support for page and per_page query params in api/search/software API endpoint
  • Amended search_software.rest test API endpoint with page and per_page query params

This is to prepare for pagination support in the frontend

Software Model

PR #210:

  • Amended software model by changing developedBy and maintainedBy fields from reference to user to an array of String

Note: This is due to the fact that the developer(s) or maintainer(s) may not be a registered user

JonathanLeeWH and others added 3 commits July 28, 2021 15:10
…oftware` API endpoint

Amended `search_software.rest` test API endpoint with `page` and `per_page` query params

This is to prepare for pagination support in the frontend
… fields from reference to `user` to an array of string
…model

Amended `software` model by changing `developedBy` and `maintainedBy` fields from reference to `user` to an array of `String`
@JonathanLeeWH JonathanLeeWH added enhancement New feature or request status.done Done and ready for review and merge priority.high High priority db.model Related to database models/schemas API Related to API endpoints labels Jul 29, 2021
@JonathanLeeWH JonathanLeeWH added this to the Version 1.0.0 milestone Jul 29, 2021
@JonathanLeeWH JonathanLeeWH self-assigned this Jul 29, 2021
@JonathanLeeWH JonathanLeeWH added the Testing Related to testing label Jul 29, 2021
@JonathanLeeWH JonathanLeeWH merged commit f9330d6 into integrate-with-frontend-web-app Jul 29, 2021
@JonathanLeeWH JonathanLeeWH deleted the improved-search-software-api-endpoint branch July 29, 2021 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Related to API endpoints db.model Related to database models/schemas enhancement New feature or request priority.high High priority status.done Done and ready for review and merge Testing Related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant