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

[feat] /services/{service}/projections/{projection} implementation (1/2) #63

Closed
11 tasks done
lgrd opened this issue Apr 4, 2023 · 0 comments · Fixed by #67
Closed
11 tasks done

[feat] /services/{service}/projections/{projection} implementation (1/2) #63

lgrd opened this issue Apr 4, 2023 · 0 comments · Fixed by #67
Assignees
Labels
enhancement New feature or request

Comments

@lgrd
Copy link
Collaborator

lgrd commented Apr 4, 2023

Need

Code the /services/{service}/projections/{projection} route of the admin API. This is its first part : return a 200 if the projection exists, 404 in the other case.

Tasks

  • Be sure of the IO inside the API yaml
  • Code it. See next paragraph.
  • Update tests
    • utest/itest if modification/Creation of a class
    • rtest if modification of an API
  • Run tests
    • Inside the docker provided by the project : rtest, ctest, utest, itest
    • Manually if necessary
  • Update the documentation
    • changelog.md
    • documentation API

Code

  • Create new request and response classes
  • Add the route inside the router
    • Controls about the the ids are made here
  • Create a function computeRequest() inside the administrateur to send the request to its serviceManager
  • Modify the service class to handle this new request
    • Modify computeAdminRequest()
    • Create the function computeProjectionRequest(), it will call the projectionManager.isProjectionLoaded(id)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants