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

KISS the serratus API #68

Open
almosnow opened this issue Oct 12, 2023 · 0 comments
Open

KISS the serratus API #68

almosnow opened this issue Oct 12, 2023 · 0 comments

Comments

@almosnow
Copy link
Contributor

As the serratus API was developed, many features were added for which API routes (and backed methods) exist, some of these overlap or are relatively complex to grasp. As an example, the frontend code that renders the RdRp explorer makes 5 API calls, the info within them is redundant and it could definitely be simplified to one (or two at most) API calls.

image

Ideally:

  • For each view on the frontend, there should be a corresponding API route that returns all the data needed to render it.
  • API routes should be simple to understand and use, /path and a list of SRA ids.
  • API routes should correspond directly to database tables or database views, this to simplify the whole data model and also to let the backend take care of most of the pre-processing needed (w/ appropriate indexes and caching).
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

No branches or pull requests

1 participant