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

CLEANUP: Remove API routes that are not used #31

Open
3 tasks
tiblu opened this issue Nov 6, 2018 · 3 comments
Open
3 tasks

CLEANUP: Remove API routes that are not used #31

tiblu opened this issue Nov 6, 2018 · 3 comments
Labels
enhancement Making an existing feature better. good first issue Good for newcomers.

Comments

@tiblu
Copy link
Member

tiblu commented Nov 6, 2018

TODO

NOTE: Verify usage against application logs!

  • REMOVE API: GET /api/users/:userId/groups/:groupId/topics, which is replaced by GET /api/users/:userId/groups/:groupId/members/topics
  • REMOVE API ROUTE: GET/POST /api/users/:userId/groups/:groupId/members which is replaced by /api/users/:userId/groups/:groupId/members/users
  • REMOVE API ROUTE: PUT/DELETE /api/users/:userId/groups/:groupId/members/:userId which is replaced by /api/users/:userId/groups/:groupId/members/users/:userId
@tiblu tiblu added enhancement Making an existing feature better. good first issue Good for newcomers. labels Nov 6, 2018
@ilmartyrk ilmartyrk assigned ilmartyrk and unassigned ilmartyrk Oct 29, 2021
@ilmartyrk
Copy link
Member

Still relevant, this list should be updated and fixed

@anettlinno
Copy link
Collaborator

Triage 37. Check API code and harvest out those end points and parts of code that we don´t use any more. Approx. dev. time 1h. Sending to development.

@BeccaMelhuish
Copy link
Contributor

@ilmartyrk will assume OK to go to 'later' as old, dormant issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Making an existing feature better. good first issue Good for newcomers.
Projects
Status: Backlog - Later
Development

No branches or pull requests

4 participants