bcgov / cas-reporting

This is for the Clean Growth Digital Services team for work related to reporting.
Apache License 2.0
0 stars 0 forks source link

API endpoint for operation list on Reporting Dashboard page #243

Open pbastia opened 1 month ago

pbastia commented 1 month ago

Description:

As a developer, I want to have an API endpoint to fetch all the data for the Operations table page, so that I can populate the page

Note: this endpoint does NOT handle pagination at this point

Acceptance Criteria:

Given I develop the front end page for the operations table for a particular user When I want to fetch the data for the table Then I have access to an API endpoint returning all that data And that endpoint is exposed on /api/v1/operations

Given I call the API endpoint for the operations page When I parse the data Then it is in JSON format And contains an array of objects with the data for the current reporting period

Data:

[
  {
    name: <string>,
    facility_count: <int>,
    report_id?: <id>
  },
..
]

Development Checklist:

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

Definition of Done (Note: If any of these points are not applicable, mark N/A)

Notes:

Blocking

234

Add report:id if exists show continue, if none show start.

patriciarussellCAS commented 1 month ago

Hey @pbastia can you confirm that this ticket is related to the "Review Operation Information" page in reporting? Is it also impacting the "Reporting Dashboard" page, where we pull in Operation names? I just want to make sure I'm understanding the term "Operation table" and "Operation page" mentioned above correctly.

pbastia commented 1 month ago

@patriciarussellCAS this is related to the operations list page (#217) and is the API endpoint supporting it. For now we just use the registration one, which doesn't expose all the necessary information

pbastia commented 2 weeks ago

@Sepehr-Sobhani can we confirm the v1 in the API path? I don't think this is part of a specific API version for now

rdromey commented 2 weeks ago

@pbastia to split out various tickets

pbastia commented 2 weeks ago

Development items were taken out and added to #234 and #219

Sepehr-Sobhani commented 2 weeks ago

@Sepehr-Sobhani can we confirm the v1 in the API path? I don't think this is part of a specific API version for now

We have been discussing the best approach for splitting our APIs and adding new features on top of existing ones. We have found that splitting routes into different folders is the best way with minimal changes to the codebase, rather than versioning the API itself. This approach allows us to separate routes based on different backend apps, and we don't have to refactor multiple frontend codes to separate the logic.