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

API route to retrieve all view entities in database #434

Open
rmainwork opened this issue Nov 2, 2021 · 1 comment
Open

API route to retrieve all view entities in database #434

rmainwork opened this issue Nov 2, 2021 · 1 comment
Labels
On Hold This will not be worked on until further notice

Comments

@rmainwork
Copy link
Contributor

This ticket implements the R part of CRUD (or GET /api/user/views). This API route should return only the views belonging to the current user. It should do this by inspecting the currently authenticated user and querying the database for view.eppn == user.eppn.

To complete this ticket, we'll probably need:

  • A ViewService with a getByUser method or similar. Into that method we can provide the user EPPN
  • Also been wondering about creating some kind of request-scoped CURRENT_USER token to @Inject into controller actions so that we can @Inject(CURRENT_USER) both here and (potentially) in other places if we want to later, rather than injecting Request and inspecting req.user each time.
@jonseitz jonseitz added the On Hold This will not be worked on until further notice label Jan 11, 2023
@jonseitz
Copy link
Contributor

As with #438, this won't be done unless/until our users ask for it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
On Hold This will not be worked on until further notice
Projects
None yet
Development

No branches or pull requests

2 participants